On Sat, Jun 19, 2021 at 10:26:40PM -0700, Dmitry Torokhov wrote: > On Sat, Jun 19, 2021 at 10:10:37PM -0700, Dmitry Torokhov wrote: > > On Wed, Jun 16, 2021 at 03:48:51PM -0700, Yizhuo Zhai wrote: > > > Inside function hideep_nvm_unlock(), variable "unmask_code" could > > > be uninitialized if hideep_pgm_r_reg() returns error, however, it > > > is used in the later if statement after an "and" operation, which > > > is potentially unsafe. > > > > I think this is pretty sensible, but let's see if the original author > > has some comments... > > I guess not. Oh well... > > Applied, thank you. Note that I had to make some changes to make it compile. Please next time try building your changes before posting them, Thanks. -- Dmitry