Re: [PATCH] afs: Fix logically dead code in afs_update_cell

2019-05-28 Thread David Howells
Marc Dionne wrote: > > diff --git a/fs/afs/cell.c b/fs/afs/cell.c > > index 9c3b07ba..980de60bf060 100644 > > --- a/fs/afs/cell.c > > +++ b/fs/afs/cell.c > > @@ -387,7 +387,6 @@ static int afs_update_cell(struct afs_cell *cell) > > if (ret == -ENOMEM) > >

Re: [PATCH] afs: Fix logically dead code in afs_update_cell

2019-05-28 Thread Marc Dionne
On Mon, May 27, 2019 at 1:54 PM Gustavo A. R. Silva wrote: > > Fix logically dead code in switch statement. > > Notice that *ret* is updated with -ENOMEM before the switch statement > at 395: > > 395 switch (ret) { > 396 case -ENODATA: > 397 case -ED

[PATCH] afs: Fix logically dead code in afs_update_cell

2019-05-27 Thread Gustavo A. R. Silva
Fix logically dead code in switch statement. Notice that *ret* is updated with -ENOMEM before the switch statement at 395: 395 switch (ret) { 396 case -ENODATA: 397 case -EDESTADDRREQ: 398 vllist->status = DNS_LOOKUP_GOT_NOT_