LINUX: Avoid duplicate mntget in afs_linux_raw_open
authorAndrew Deason <adeason@sinenomine.net>
Wed, 17 Apr 2013 06:33:07 +0000 (01:33 -0500)
committerDerrick Brashear <shadow@your-file-system.com>
Wed, 17 Apr 2013 12:36:27 +0000 (05:36 -0700)
commit5ccbbda19f11e7027300409c46715155f439424a
tree3f23dfdbcdff81a7bb1c594a6ff30063530fd5be
parente019429d4548348c6ab8674305d985feee040476
LINUX: Avoid duplicate mntget in afs_linux_raw_open

In the unlikely event that our afs_dentry_open call fails with
cache_creds, we call afs_dentry_open again with the current creds as a
fallback. However, we call mntget on afs_cacheMnt for each call. So if
we actually hit the second call, we'll have added 2 refs to
afs_cacheMnt, but we only actually opened one file, causing a slight
overcount on afs_cacheMnt refs.

To avoid this, just call mntget once, before any of the
dentry_open-related calls.

Change-Id: I7ec3e8c193dd7782ab629fb5d7615d83f8385b6c
Reviewed-on: http://gerrit.openafs.org/9791
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Marc Dionne <marc.c.dionne@gmail.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
src/afs/LINUX/osi_file.c