segfault in mount with CITI patch

[Vince Busam <vbusam@google.com>]

I've found that mount (from util-linux 2.12 + CITI_NFS4_ALL-3 patch) will segfault with 
this (probably rare) command:

mount -o tcp,nosuid,rw,rsize=8192,wsize=8192,nfsvers=3,mountvers=2,intr,sec=krb5 
server:/volume /mntpoint

This is because nfs_call_mount() will run nfs2_mount() when mountvers=2, but since 
nfsvers=3, nfsmount() will expect certain fields to be filled out, which are not.  Causing 
a segfault here, where flavor points to an invalid address.
1061                            if (flavor[i] == data.pseudoflavor)


Is nfsvers=3,mountvers=2 even OK to do?  If not, mount should print an error instead of 
segfault.

Thanks,
Vince


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs