Re: [edk2-devel] [PATCH 1/1] MdeModulePkg/Bus/Ata/AtaBusDxe: Fix SIGN_EXTENSION Coverity issue

2023-06-08 Thread Ranbir Singh
Yes, I have already noted to update this patch on the same lines as discussed in context of https://edk2.groups.io/g/devel/topic/99293622 On Thu, Jun 8, 2023 at 12:48 PM Wu, Hao A wrote: > > -Original Message- > > From: Ranbir Singh > > Sent: Saturday, June 3, 2023 12:09 AM > > To:

Re: [edk2-devel] [PATCH 1/1] MdeModulePkg/Bus/Ata/AtaBusDxe: Fix SIGN_EXTENSION Coverity issue

2023-06-08 Thread Wu, Hao A
> -Original Message- > From: Ranbir Singh > Sent: Saturday, June 3, 2023 12:09 AM > To: devel@edk2.groups.io; rsi...@ventanamicro.com > Cc: Wu, Hao A ; Ni, Ray > Subject: [PATCH 1/1] MdeModulePkg/Bus/Ata/AtaBusDxe: Fix > SIGN_EXTENSION Coverity issue > > From: Ranbir Singh > > Line

[edk2-devel] [PATCH 1/1] MdeModulePkg/Bus/Ata/AtaBusDxe: Fix SIGN_EXTENSION Coverity issue

2023-06-02 Thread Ranbir Singh
From: Ranbir Singh Line number 365 does contain a typecast with UINT32, but it is after all the operations (16-bit left shift followed by OR'ing) are over. To avoid any SIGN_EXTENSION, typecast the intermediate result after 16-bit left shift operation immediately with UINT32. Cc: Hao A Wu Cc: