On 4/17/23 13:41, Christophe JAILLET wrote: > 'status' is known to be 0 at the point. > And nvmet_auth_challenge() return a -E<ERROR_CODE> or 0. > So these lines of code should just be removed. > > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > --- > The dead code became obvious after commit be2ada6d0ed0 ("nvmet-auth: fix > return value check in auth receive") > --- > Looks good. Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx> -ck