mountd: Unregister mountd if my_svc_run() returns

Fix a long standing bug: when my_svc_run() returns, mountd should
unregister itself with the local rpcbind so that it can subsequently
start cleanly.

Log a more helpful error message in this case.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>

diff --git a/utils/mountd/mountd.c b/utils/mountd/mountd.c
index a4fdcc8a98082decf805e826b2a039ecfb962b83..9c7caf96e388fa433b1e3f299a4e25afa4977a8c 100644 (file)
--- a/utils/mountd/mountd.c
+++ b/utils/mountd/mountd.c
@@ -888,7 +888,8 @@ main(int argc, char **argv)
 
        my_svc_run();
 
 
        my_svc_run();
 

-       xlog(L_ERROR, "Ack! Gack! svc_run returned!\n");
+       xlog(L_ERROR, "RPC service loop terminated unexpectedly. Exiting...\n");
+       unregister_services();

        exit(1);
 }
 
        exit(1);
 }