> Looking at the reply code just above this, discovered never executed code
> section.  I'm surprised the compiler isn't giving a warning.
> 
> Checking the Lieb re-indent commit, there was a label handle_err.
> 
> I'll go ahead and remove it.  It merely seems to be old logging....
> Just a head's up for somebody to remember a need for this code.

I looked at the patch that removed the handle_err label. I'm not sure why I 
didn't remove the rest of this code at that time, but obviously it is dead 
code. Thanks for spotting it.

Frank

> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1411)
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1412)     /* 
> Finish any
> request not already deleted */
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1413)     if 
> (dpq_status
> == DUPREQ_SUCCESS)
> 00c21a6878 (William Allen Simpson 2015-06-12 07:36:46 -0400 1414)
>       dpq_status = nfs_dupreq_finish(&reqdata->r_u.req.svc, res_nfs);
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1415)     goto 
> freeargs;
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1416)
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1417)     /* 
> Reject the
> request for authentication reason (incompatible
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1418)      * file 
> handle)
> */
> 822fdbc104 (Frank S. Filz         2014-03-20 15:01:31 -0400 1419)     if
> (isInfo(COMPONENT_DISPATCH) || isInfo(COMPONENT_EXPORT)) {
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1420)             
> char
> dumpfh[1024];
> 33be2e2c11 (Frank S. Filz         2015-05-19 18:41:15 -0400 1421)
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1422)
>       sprint_fhandle3(dumpfh, (nfs_fh3 *) arg_nfs);
> 822fdbc104 (Frank S. Filz         2014-03-20 15:01:31 -0400 1423)
>       LogInfo(COMPONENT_DISPATCH,
> 66a09b1375 (Frank S. Filz         2017-12-07 11:30:32 -0800 1424)
>       "%s Request from host %s V3 not allowed on this export, proc=%"
> 66a09b1375 (Frank S. Filz         2017-12-07 11:30:32 -0800 1425)
>       PRIu32 ", FH=%s",
> ba42f03b2c (Frank S. Filz         2014-05-19 14:23:01 -0400 1426)
>       progname, client_ip,
> eb51d7a97b (William Allen Simpson 2017-01-13 19:19:55 -0500 1427)
>               reqdata->r_u.req.svc.rq_msg.cb_proc, dumpfh);
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1428)     }
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1429)     auth_rc 
> =
> AUTH_FAILED;
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1430)
> 02526d7325 (Jim Lieb              2013-10-10 20:50:47 -0700 1431)  
> auth_failure:



------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to