Hello Denis,

As far as I can recall, the fix you mentioned is a change that to please the 
static analysis tool.
My opinion is that it not a critical fix.

Best Regards,
Hao Wu

> -----Original Message-----
> From: Denis Nikitin <[email protected]>
> Sent: Thursday, January 6, 2022 3:19 AM
> To: Wu, Hao A <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> Ramasubramanian, Karthik <[email protected]>
> Subject: Change from "BaseTools/LzmaCompress: Fix possible uninitialized
> variable" patch was reverted
> 
> Hi Hao,
> 
> While updating edk2 in Chrome OS we noticed that the change in your patch
> https://edk2.groups.io/g/devel/message/19270 was reverted in
> commit:
> 5ec5a236d1 BaseTools Lzma: Update LZMA SDK version to 18.05.
> 
> If you think the fix is critical I think it should be merged in https://www.7-
> zip.org/sdk.html.
> 
> Thanks,
> Denis


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#85294): https://edk2.groups.io/g/devel/message/85294
Mute This Topic: https://groups.io/mt/88221936/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to