]> git.decadent.org.uk Git - nfs-utils.git/commit
mountd: Make local functions static
authorChuck Lever <chuck.lever@oracle.com>
Tue, 30 Oct 2012 18:06:22 +0000 (14:06 -0400)
committerSteve Dickson <steved@redhat.com>
Tue, 30 Oct 2012 19:30:08 +0000 (15:30 -0400)
commit15b940351e88da5cb3d68d71887196f89a13e0de
tree3a2765cf409808ea9ad31a2930663ba8374aaa2d
parenta543ffd62ceade54ad0663765ae2909b6be361f9
mountd: Make local functions static

Removed a couple Wmissing-prototypes warnings in the mountd code.

Once the parse_fsid() function was made static, the compiler
detected execution paths through it that did not initialize some
fields in *parsed.

[ I'm pretty sure these problems are currently harmless, since each
path is taken depending on the value of the .fsidtype field.  Each
path accesses only the fields in *parsed that it cares about. ]

This is because parsed_fsid isn't a union type.  parse_fsid() leaves
uninitialized fields that are not used by a particular fsidtype.  To
prevent an accidental dereference of stack garbage (.fhuuid being an
example of a pointer that is left uninitialized sometimes), have
parse_fsid() defensively pre-initialize *parsed to zero.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
utils/mountd/cache.c