Tianocore/edk2.git/RedfishPkg, branch dependabot/github_actions/github/codeql-action-4EDK II (mirror)https://git.radix-linux.su/Tianocore/edk2.git/atom?h=dependabot%2Fgithub_actions%2Fgithub%2Fcodeql-action-42025-11-02T13:10:47+00:00RedfishPkg: Add missing FreePool to fix memory leak issue2025-11-02T13:10:47+00:00Qihang Gaogaoqihang@loongson.cn2025-10-28T03:22:12+00:00urn:sha1:2ff1029cc38ad1b9955c57c5aa04ce0e2127cb9d
According to the implementation of `HiiGetSupportedLanguages`, the caller
is responsible for freeing the returned string using FreePool().
Signed-off-by: Qihang Gao <gaoqihang@loongson.cn>
RedfishPkg: Drop ARM32 Support2025-09-25T22:04:10+00:00Oliver Smith-Dennyosde@microsoft.com2025-09-15T14:55:14+00:00urn:sha1:470a80094c7bea460cfb66d2b804d7404ada3345
edk2 is dropping support for the ARM32 architecture. This
commit drops ARM32 code from RedfishPkg. This also drops
irrelevant VALID_ARCHITECTURES comments in infs that are
not arch specific.
Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
RedfishPkg/JsonLib: Define NO_MSABI_VA_FUNCS for GCC X642025-09-19T10:54:10+00:00Michael D Kinneymichael.d.kinney@intel.com2025-09-05T00:03:44+00:00urn:sha1:82a03a824824a348fbbff7de680e15284bfc61a4
For GCC family X64 builds, add -DNO_MSABI_VA_FUNCS to force
use of the compiler default vararg macros. This define is
required in libraries that use a standard C source submodule
and there are one or more vararg functions. The calling
convention of the vararg function must match the calling
convention of the vararg macros used within the vararg
function.
In the JsonLib case, jsonp_error_set() is a vararg function
and generates the following error for CLANGDWARF X64 builds.
RedfishPkg/Library/JsonLib/jansson/src/error.c:37:5:
error: '__builtin_ms_va_start' used in System V ABI function
37 | va_start(ap, msg);
| ^
This change matches CryptoPkg openssl and mbedtls library
INF files that also define NO_MSABI_VA_FUNCS.
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
RedfishPkg/PlatformConfig: Use en-US if no x-uefi-redfish string2025-08-27T22:21:25+00:00Abner Changabner.chang@amd.com2025-08-18T16:49:14+00:00urn:sha1:4b041f09d62ff569b2d3e1829387992de7cf237a
Search en-US HII value string if x-uefi-redfish language of the
value string is not found. Because not all of the HII
value strings have the x-uefi-redfish language string define,
such as "Enabled" or "Disabled".
Signed-off-by: Abner Chang <abner.chang@amd.com>
RedfishPkg/RedfishPlatformConfig: Expose suppressed HII options to Redfish2025-08-27T15:16:26+00:00Abner Changabner.chang@amd.com2025-06-05T07:34:00+00:00urn:sha1:20609b499e67ada9c883d7d9bfbf6ac09df2040c
When REDFISH_PLATFORM_CONFIG_ALLOW_SUPPRESSED is set in
PcdRedfishPlatformConfigFeatureProperty, HII suppressed
options should be exposed to Redfish BIOS resource.
Signed-off-by: Abner Chang <abner.chang@amd.com>
RedfishPkg: Correct spelling errors and typos2025-07-04T01:41:47+00:00Abdul Lateef AttarAbdulLateef.Attar@amd.com2025-07-01T09:04:51+00:00urn:sha1:96ce01e6e7770175639d00c0ba5e5dfa880e389f
Signed-off-by: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
RedfishPkg/JsonLib: Use same defines for MSFT and GCC families2025-06-13T15:47:59+00:00Michael D Kinneymichael.d.kinney@intel.com2025-05-19T20:21:15+00:00urn:sha1:02e967f1e462a540f430b77248516322189e9b63
CLANGPDB and CLANGDWARF are in GCC family. When CLANGPDB or
CLANGDWARF are used from windows host, the same defines and
undefines must be applied to GCC family for CLANG builds.
This resolves CLANG 20.1.0 build errors.
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
RedfishPkg/RedfishLib: Fix enum type mismatch2025-06-13T15:47:59+00:00Michael D Kinneymichael.d.kinney@intel.com2025-05-19T20:20:14+00:00urn:sha1:6c1fd8d56776ca54c2f43b9c94f297320d6424eb
Fix enum type mismatch between jansson library and JsonLib.
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
RedfishPkg/RedfishConfigHandler: Free handle buffers2025-06-12T14:09:13+00:00Dongyan Qianqiandongyan@loongson.cn2025-06-11T03:58:13+00:00urn:sha1:108757b613620e8406f32e5aa2e52df8d115d61f
Fix memory leaks by adding missing FreePool calls:
free handle buffers in Stop and initialization routines.
REF:https://uefi.org/sites/default/files/resources/UEFI_Spec_Final_2.11.pdf
Chapter 7.3.15: "Services - Boot Services.LocateHandleBuffer":
It is the caller's responsibility to call the Boot Service.FreePool when
the caller no longer requires the contents of Buffer.
Signed-off-by: Dongyan Qian <qiandongyan@loongson.cn>
RedfishPkg: Don't define bool type if building in C23 mode2025-05-29T23:15:20+00:00Rebecca Cranrebecca@bsdio.com2025-05-26T14:07:41+00:00urn:sha1:4518ba2a2b03b6c8071f51b3bee360b933fcb441
In C23 bool is a built-in type, so it's not necessary to typedef
bool in RedfishCrtLib.h.
Signed-off-by: Rebecca Cran <rebecca@bsdio.com>