NLM: Set address family before calling nlm_host_rebooted()
authorChuck Lever <chuck.lever@ORACLE.COM>
Thu, 23 Oct 2008 04:50:35 +0000 (00:50 -0400)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Thu, 30 Oct 2008 21:19:30 +0000 (17:19 -0400)
The nlm_host_rebooted() function uses nlm_cmp_addr() to find an
nsm_handle that matches the rebooted peer.  In order for this to work,
the passed-in address must have a proper address family.

This fixes a post-2.6.28 regression introduced by commit 781b61a6, which
added AF_INET6 support to nlm_cmp_addr().  Before that commit,
nlm_cmp_addr() didn't care about the address family; it compared only
the sin_addr.s_addr field for equality.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
fs/lockd/svc4proc.c
fs/lockd/svcproc.c

index 014f6ce48172fa73cc5d534bda12096776b39e20..4dfdcbc6bf6858e626ad99218a0b2abf254424b4 100644 (file)
@@ -434,6 +434,7 @@ nlm4svc_proc_sm_notify(struct svc_rqst *rqstp, struct nlm_reboot *argp,
         * reclaim all locks we hold on this server.
         */
        memset(&saddr, 0, sizeof(saddr));
+       saddr.sin_family = AF_INET;
        saddr.sin_addr.s_addr = argp->addr;
        nlm_host_rebooted(&saddr, argp->mon, argp->len, argp->state);
 
index 548b0bb2b84d1e7dccb6351bdff03980d276959e..3ca89e2a93812d32736f76ba3c444f28bc43a602 100644 (file)
@@ -466,6 +466,7 @@ nlmsvc_proc_sm_notify(struct svc_rqst *rqstp, struct nlm_reboot *argp,
         * reclaim all locks we hold on this server.
         */
        memset(&saddr, 0, sizeof(saddr));
+       saddr.sin_family = AF_INET;
        saddr.sin_addr.s_addr = argp->addr;
        nlm_host_rebooted(&saddr, argp->mon, argp->len, argp->state);