Fix parsing single section CPER error

Single section CPER logs do not contain a Record Header. The check for
a valid header is now performed only if the content includes a Record
Header. Additionall

Fix parsing single section CPER error

Single section CPER logs do not contain a Record Header. The check for
a valid header is now performed only if the content includes a Record
Header. Additionally, this update fixes an issue with the size
calculation for single-section logs.

Tested :
$> ./cper-generate --out single-cper.file --single-section firmware
$> ./cper-convert to-json-section single-cper.file

Change-Id: Id7efec20df3e5b53c6f90330e0770c1f682a8d87
Signed-off-by: John Chung <john.chung@arm.com>

show more ...

Fix print qualifier for readsize property

This was causing compile errors in yocto, fixed with correct type.

Change-Id: I06528391e7dab9d30af2577237ccfc84132823cf
Signed-off-by: Aushim Nagarkatti <a

Fix print qualifier for readsize property

This was causing compile errors in yocto, fixed with correct type.

Change-Id: I06528391e7dab9d30af2577237ccfc84132823cf
Signed-off-by: Aushim Nagarkatti <anagarkatti@nvidia.com>

show more ...

Allow parsing base64 files

Redfish outputs base64 strings. It would be useful if the CLI app could
read those strings in directly.

This commit breaks out a new method "header_valid" to allow tooli

Allow parsing base64 files

Redfish outputs base64 strings. It would be useful if the CLI app could
read those strings in directly.

This commit breaks out a new method "header_valid" to allow tooling to
do an initial reading of a buffer to determine if it appears to be
correct before going further. This allows the CLI app to attempt to
parse as a buffer, if that fails, attempt to parse as base64.

To support as many inputs as possible, this commit makes padding
optional. It also allows a trailing \n as is present in many files.

Change-Id: I4fb759ecefc8ce1c757f1a9e7c4a2b2d220105d0
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

Use less c++

As a design, it would be better if we didn't need to have a dependency
on c++ in a c library. Moving things to C will reduce both
dependencies and compile times.

Reduced dependencies

Use less c++

As a design, it would be better if we didn't need to have a dependency
on c++ in a c library. Moving things to C will reduce both
dependencies and compile times.

Reduced dependencies makes the library itself easier to use in
more environments.

libmctp Is a library that doesn't take a dependency on gtest for its
tests
libpldm Is an example that does take a dependency on gtest.

Change-Id: If1dc638f87b75b28181a0baf67f5a18d389cff81
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

Clean up properly in cper-convert

This could potentially leave a json_object around, fix it.

Change-Id: I9a413cdb57ea23a5450691f535f08010c635a9ad
Signed-off-by: Ed Tanous <etanous@nvidia.com>

Implement common logging function

When used as a library, it's desirable to be able to suppress logging,
or pipe logging through a different path. This commit changes behavior
such that logging is

Implement common logging function

When used as a library, it's desirable to be able to suppress logging,
or pipe logging through a different path. This commit changes behavior
such that logging is disabled by default, and introduces 2 new methods,
cper_set_log_stdio and cper_set_log_custom.

These allow library integrators to specify their logging mode. In
practice, this also allows fuzzing to run faster by not printing errors
to the log.

Change-Id: I941476627bc9b8261ba5f6c0b2b2338fdf931dd2
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

Remove json schema validation

libcper is a cper parsing library. It is not a json schema validation
library. If we want to add support for this in the CLI, we should use a
relevant library. The j

Remove json schema validation

libcper is a cper parsing library. It is not a json schema validation
library. If we want to add support for this in the CLI, we should use a
relevant library. The json schema validation that's here isn't correct
to the json spec, to the point where it needed to be replaced in unit
tests. Adding internal json schema validation doesn't add value here.
It is trivial to run jsonschema as a cli app, there's not a lot of
reason to keep it here and take up the binary space.

Change-Id: I076c8eb9f81c34cdfc40525fe940e8fba0f945b3
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

Change include style to use system includes

The libcper header files in `libcper` are installed to
`usr/include/libcper`. Use that system includes in `libcper` source
instead of using the project in

Change include style to use system includes

The libcper header files in `libcper` are installed to
`usr/include/libcper`. Use that system includes in `libcper` source
instead of using the project includes.

Change-Id: I596edc2c754dae4829844f535a1e34caa246fb43
Signed-off-by: Thu Nguyen <thu@os.amperecomputing.com>

show more ...

Update limits headers for better platform compat

This change enables building libcper based cli tools on mac systems.

Change-Id: If9b0134d3677b84c57603fa83d546b58e8c2d073
Signed-off-by: Andrew Adri

Update limits headers for better platform compat

This change enables building libcper based cli tools on mac systems.

Change-Id: If9b0134d3677b84c57603fa83d546b58e8c2d073
Signed-off-by: Andrew Adriance <aadriance@nvidia.com>

show more ...

Remove trailing whitespace

clang-format won't remove trailing whitespace if that's the only change.
Fix them all.

Change-Id: Ic6e14af43cdd11905d3b58430d49b9ec1484f812
Signed-off-by: Ed Tanous <ed@t

Remove trailing whitespace

clang-format won't remove trailing whitespace if that's the only change.
Fix them all.

Change-Id: Ic6e14af43cdd11905d3b58430d49b9ec1484f812
Signed-off-by: Ed Tanous <ed@tanous.net>

show more ...

Formatting .c/.h files and fix memory leakage issues

Signed-off-by: John Chung <john.chung@arm.com>
Change-Id: Id8328f412c2724992d80c0b3f895c8f85bc4ae68

Source json.h non-locally.

Change-Id: Ia42c41fde74596b394a1f7ae0021f1a5510991a8

Add CPER conversion for single section records.

Fix minor generation bug, reformat CLI parsing.

Fix several review comments.

Partial reformat to kernel code style.

Add test suite, fix a mountain of small errors.

Add "cper-generate", update README, fix minor bugs.