]> git.decadent.org.uk Git - nfs-utils.git/commitdiff
v4_root_add_parents: remove a possible buffer overflow.
authorNeilBrown <neilb@suse.de>
Tue, 1 May 2012 19:26:54 +0000 (15:26 -0400)
committerSteve Dickson <steved@redhat.com>
Tue, 1 May 2012 19:30:00 +0000 (15:30 -0400)
The loop in v4root_add_parents() is a little odd.
The first time through, 'ptr' points immediately "beyond"
a '/' character (the first).  For every other iterration it points
directly "at" a '/' character.
Such inconsistency is error prone and infact there is an error.
If "path" is precisely "/", then the first call to
   ptr = strchr(ptr, '/')

will be given a 'ptr' which is beyond the '\0' at the end of
"path".  This could potentially contain anything and the strchr()
could search well beyond a buffer (though this depends on exactly how
the string is set up which depends on separate code).

So change the loop to have 'ptr' always point at a '/', and
handle the special case of "/" explicitly.

Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Steve Dickson <steved@redhat.com>

No differences found