[AMD Official Use Only - General] Hi Nickle, As this change is in the autogenerate source files. We have to update the script here: https://github.com/DMTF/Redfish-Schema-C-Struct-Generator Not sure if you have access to this repo, however I am pleased inviting you to update the script on DMTF Github. We will not merging this change on edk2-redfish-client repo.
Thanks Abner > -----Original Message----- > From: Nickle Wang <nick...@nvidia.com> > Sent: Tuesday, February 27, 2024 8:53 PM > To: devel@edk2.groups.io > Cc: Chang, Abner <abner.ch...@amd.com>; Igor Kulchytskyy > <ig...@ami.com>; Nick Ramirez <nrami...@nvidia.com> > Subject: [edk2-redfish-client][PATCH] RedfishClientPkg/ConverterLib: check > JSON value type > > Caution: This message originated from an External Source. Use proper caution > when opening attachments, clicking links, or responding. > > > Check and see if JSON object type is desired type before > getting its value. According to the Redfish schema, attribute > value can be null value. Add this error handling to avoid > system assertion while getting attribute value. > > Signed-off-by: Nickle Wang <nick...@nvidia.com> > Cc: Abner Chang <abner.ch...@amd.com> > Cc: Igor Kulchytskyy <ig...@ami.com> > Cc: Nick Ramirez <nrami...@nvidia.com> > --- > RedfishClientPkg/ConverterLib/src/RedfishCsCommon.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/RedfishClientPkg/ConverterLib/src/RedfishCsCommon.c > b/RedfishClientPkg/ConverterLib/src/RedfishCsCommon.c > index 250ef75e4..7ee3c86e1 100644 > --- a/RedfishClientPkg/ConverterLib/src/RedfishCsCommon.c > +++ b/RedfishClientPkg/ConverterLib/src/RedfishCsCommon.c > @@ -1,5 +1,7 @@ > /** @file > > + Copyright (c) 2024, NVIDIA CORPORATION & AFFILIATES. All rights > reserved. > + > Copyright Notice: > Copyright 2019-2024 Distributed Management Task Force, Inc. All rights > reserved. > License: BSD 3-Clause License. For full text see link: > https://github.com/DMTF/Redfish-JSON-C-Struct- > Converter/blob/master/LICENSE.md > @@ -829,6 +831,10 @@ GetRedfishPropertyStr ( > return RedfishCS_status_not_found; > } > > + if (!json_is_string (TempJsonObj)) { > + return RedfishCS_status_not_found; > + } > + > Status = allocateDuplicateStr (Cs, (char *)json_string_value (TempJsonObj), > (void **)DstBuffer); > return Status; > } > @@ -914,6 +920,10 @@ GetRedfishPropertyInt64 ( > return Status; > } > > + if (!json_is_integer (TempJsonObj)) { > + return RedfishCS_status_not_found; > + } > + > **Dst = (RedfishCS_int64)json_integer_value (TempJsonObj); > return RedfishCS_status_success; > } > -- > 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#116070): https://edk2.groups.io/g/devel/message/116070 Mute This Topic: https://groups.io/mt/104601599/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-