Skip to content

Changes

Summary

  1. gitignore: Set LC_COLLATE=C sort standard for file (details)
  2. gitignore: Add default Eclipse project bin/ folder (details)
  3. README: Fix the few markdownlint warnings 009, 012 (details)
  4. FtraceTestTraceTest: Remove unused import (details)
  5. Apply default Eclipse formatting for .java files (details)
  6. Add invalid trace files for ftrace security testing (details)
  7. Rename binary ftrace test trace recorded on a multiple-core machine (details)
Commit 724ef8cfc97293f438bd00af2dd118b834d5962f by Matthew Khouzam
gitignore: Set LC_COLLATE=C sort standard for file

Remove the usually trivial line comments about these being from Eclipse.

Change-Id: I206a4b636ea8b54077cb12dc10e008cb4e601809
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/tracecompass-test-traces/+/190365
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
The file was modified .gitignore (diff)
Commit b45546defa68a078ceca20e3b0cf72245ab4437c by Matthew Khouzam
gitignore: Add default Eclipse project bin/ folder

Change-Id: Ic3f8444c92b4ca8a5d2a7a956af4963a284ff076
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/tracecompass-test-traces/+/190366
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
The file was modified .gitignore (diff)
Commit 90008b70f2333af75e71fb4173eeda5095c86fb4 by Matthew Khouzam
README: Fix the few markdownlint warnings 009, 012

In VS Code that is.

[1] https://github.com/DavidAnson/markdownlint/blob/v0.25.1/doc/Rules.md#md009
[2] https://github.com/DavidAnson/markdownlint/blob/v0.25.1/doc/Rules.md#md012

Change-Id: I4f5e9624cc6e9ce8cab2c8b48b57d35d53e063b2
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/tracecompass-test-traces/+/190371
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
The file was modified README.md (diff)
Commit 003541c18bd7a6e1e3b69d7afcf2da5fdac15bf2 by Matthew Khouzam
FtraceTestTraceTest: Remove unused import

As flagged by an Eclipse warning.

Change-Id: I41e7939d38520870c2b115dd450850f70344427b
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/tracecompass-test-traces/+/190373
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
The file was modified ftrace/src/test/java/org/eclipse/tracecompass/testtraces/ftrace/FtraceTestTraceTest.java (diff)
Commit c7272f2c32a65f15ede7dabcfc4dad91329d8e28 by Matthew Khouzam
Apply default Eclipse formatting for .java files

While setting the automatic indentation policy to spaces-only (no tabs).

Change-Id: I3b47ab076c2d2a63a10aa8cfffab0d476cb26892
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/tracecompass-test-traces/+/190374
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
The file was modified ftrace/src/main/java/org/eclipse/tracecompass/testtraces/ftrace/FtraceTestTrace.java (diff)
The file was modified ftrace/src/test/java/org/eclipse/tracecompass/testtraces/ftrace/FtraceTestTraceTest.java (diff)
The file was modified ctf/src/main/java/org/eclipse/tracecompass/testtraces/ctf/CtfTestTrace.java (diff)
Commit 4af5a2a49a7e9a35432b9657ef0803a1a0ee11ac by Matthew Khouzam
Add invalid trace files for ftrace security testing

This commit adds 2 new binary ftrace files. In most cases, each section
in the header of binary ftrace files contains a section header, and a
section size. The size of the section allows the parser to read and
process the section data.

The 2 added ftrace files are injected with bad data, specifically bad
section sizes for the header page section and the first cpu section.
They are used for security tests and prevention of injection attacks. If
the parser uses bad section sizes to read section data, it might go out
of the file size limit and get into unauthorized memory locations.

Add the missing blank javadoc lines in FtraceTestTrace, so the default
Eclipse formatter does not do so while adding extra trailing space
characters to these (wrongfully).

Change-Id: Ia557b83416de83d0751cd33fef031bf4db09c0ef
Signed-off-by: Hoang Thuan Pham <hoang.pham@calian.ca>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/tracecompass-test-traces/+/190075
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
The file was addedftrace/src/main/resources/2.6/trace_injected_cpu.dat
The file was addedftrace/src/main/resources/2.6/trace_injected_header.dat
The file was modified ftrace/src/main/java/org/eclipse/tracecompass/testtraces/ftrace/FtraceTestTrace.java (diff)
Commit ce1c215c4277359e731dceb05732df04ee7a8fda by Matthew Khouzam
Rename binary ftrace test trace recorded on a multiple-core machine

This commit renames the valid binary ftrace test trace that was recorded
on a multiple-core machine. Previously, the trace name was
TEST_2_6_4_CPU, which was confusing for readers since it can be
understood as FTrace version 2.6.4. However, the FTrace version was 2.6
and the number of CPUs was 4. It is now renamed to
TEST_2_6_MULTIPLE_CPUS, which reflects clearly that it was recorded on a
machine with multiple cores.

Change-Id: I228512fa4d1d87d34b8551f374ad560b038b741e
Signed-off-by: Hoang Thuan Pham <hoang.pham@calian.ca>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/tracecompass-test-traces/+/190449
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
The file was modified ftrace/src/main/java/org/eclipse/tracecompass/testtraces/ftrace/FtraceTestTrace.java (diff)