git: sbin/hammer: Remove meaningless sscanf for "@@PFS%d" format
Tomohiro Kusumi
tkusumi at crater.dragonflybsd.org
Thu Oct 6 14:28:58 PDT 2016
commit 143313913f4251064142972c3a6c8b94660b4fab
Author: Tomohiro Kusumi <kusumi.tomohiro at gmail.com>
Date: Wed Sep 21 10:48:39 2016 +0900
sbin/hammer: Remove meaningless sscanf for "@@PFS%d" format
This isn't necessary as userspace never reads "@@PFS%d" via readlink(2).
"@@%jx:%d" covers all the necessary patterns.
It also shouldn't be here because the first stat(2) call in this
function ignores the result due to possible slave PFS (before once
synced with a master PFS), so an arbitrary string that matches this
"@@PFS%d" pattern for path argument (whether it really exists or not)
makes this function return non negative fd while it should return -1.
If "@@PFS%d" is the name of an existing file or a directory, then
that's also not what this function is looking for, so it can still
be ignored.
Summary of changes:
sbin/hammer/cmd_pfs.c | 24 +++++++++++-------------
1 file changed, 11 insertions(+), 13 deletions(-)
http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/143313913f4251064142972c3a6c8b94660b4fab
--
DragonFly BSD source repository
More information about the Commits
mailing list