fix rpc.gssd -d

Fix an off-by-one in the handling of the -d option to rpc.gssd; this makes the option usable again (it would complain that the string was too long). This patch is originally by Liam Bedford, and extracted from

https://bugs.launchpad.net/ubuntu/+source/nfs-utils/+bug/76409

utils/gssd/gssd.c +1 -1

@@ -118,7 +118,7 @@ main(int argc, char *argv[])
                                break;
                        case 'd':
                                strncpy(ccachedir, optarg, sizeof(ccachedir));
-                               if (ccachedir[sizeof(ccachedir-1)] != '\0')
+                               if (ccachedir[sizeof(ccachedir)-1] != '\0')
                                        errx(1, "ccachedir path name too long");
                                break;
                        default:

/* Steinar */
--
Homepage: http://www.sesse.net/

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

Thanks Steinar. I'll get to this and the AM_MAINTAINER_MODE patch soon and resend them to Neil. (Your other recent patches are out of my arena.)

Quoted 3 lines · Kevin Coffman

Thanks Steinar. I'll get to this and the AM_MAINTAINER_MODE patch soon and resend them to Neil. (Your other recent patches are out of my arena.)

Thanks.

By the way, I checked out the git repository (finding out the URL was non-trivial, though!) and discovered that the bzero patch was not applied (I believe it was sent in quite a while ago). Neil, did you forget, is it in a queue somewhere, or did I just check out the wrong one? (I used
git://git.linux-nfs.org/pub/linux/nfs-utils/.)

Quoted 5 lines · On Friday January 5, [email protected] wrote:

Quoted 3 lines · Kevin Coffman

Thanks Steinar. I'll get to this and the AM_MAINTAINER_MODE patch soon and resend them to Neil. (Your other recent patches are out of my arena.)

Quoted 7 lines

Thanks.

By the way, I checked out the git repository (finding out the URL was non-trivial, though!) and discovered that the bzero patch was not applied (I believe it was sent in quite a while ago). Neil, did you forget, is it in a queue somewhere, or did I just check out the wrong one? (I used
git://git.linux-nfs.org/pub/linux/nfs-utils/.)

git://linux-nfs.org/nfs-utils
should be enough

I don't remember a bzero patch. Is it just replacing the bzero in support/nfs/svc_socket.c with a memset ??

Thanks Steinar. I'll get to this and the AM_MAINTAINER_MODE patch soon and resend them to Neil. (Your other recent patches are out of my arena.)

K.C.

Quoted 22 lines · Steinar H. Gunderson

Fix an off-by-one in the handling of the -d option to rpc.gssd; this makes the option usable again (it would complain that the string was too long). This patch is originally by Liam Bedford, and extracted from

https://bugs.launchpad.net/ubuntu/+source/nfs-utils/+bug/76409

utils/gssd/gssd.c +1 -1

@@ -118,7 +118,7 @@ main(int argc, char *argv[])
                                break;
                        case 'd':
                                strncpy(ccachedir, optarg, sizeof(ccachedir));
-                               if (ccachedir[sizeof(ccachedir-1)] != '\0')
+                               if (ccachedir[sizeof(ccachedir)-1] != '\0')
                                        errx(1, "ccachedir path name too long");
                                break;
                        default:

/* Steinar */
--
Homepage: http://www.sesse.net/

Quoted 10 lines

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs