jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-01-31 05:27:44 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

volser: Check vnode length on dump

Browse Source 
Commit aadf69eabb1962496fa93745ab560a5b48cacd61 adds length checks on
vnodes during fileserver read/write operations. Do the same thing when
we dump volume data from the volserver, to ensure that we don't
transmit incorrect data e.g. to other RO sites when releasing.

FIXES 131530

Reviewed-on: http://gerrit.openafs.org/8836
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
(cherry picked from commit eb938bd18d9806f3529b7a2cbccdce5340c0e39f)

Change-Id: Ia49e7310f2e143d4eed965a5090e687be55864e4
Reviewed-on: http://gerrit.openafs.org/9505
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
This commit is contained in:
Andrew Deason 2012-12-14 15:05:53 -06:00 committed by Stephan Wiesand
parent c40eb13d01
commit e2f29a79a7
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/volser/dumpstuff.c
View File

@ -1087,6 +1087,7 @@ DumpVnode(struct iod *iodp, struct VnodeDiskObject *v, int volid,
VAclDiskSize(v));
}
if (VNDISK_GET_INO(v)) {
afs_sfsize_t indexlen, disklen;
IH_INIT(ihP, iodp->device, iodp->parentId, VNDISK_GET_INO(v));
fdP = IH_OPEN(ihP);
if (fdP == NULL) {
@ -1094,6 +1095,17 @@ DumpVnode(struct iod *iodp, struct VnodeDiskObject *v, int volid,
IH_RELEASE(ihP);
return VOLSERREAD_DUMPERROR;
}
VNDISK_GET_LEN(indexlen, v);
disklen = FDH_SIZE(fdP);
if (indexlen != disklen) {
FDH_REALLYCLOSE(fdP);
IH_RELEASE(ihP);
Log("DumpVnode: volume %lu vnode %lu has inconsistent length "
"(index %lu disk %lu); aborting dump\n",
(unsigned long)volid, (unsigned long)vnodeNumber,
(unsigned long)indexlen, (unsigned long)disklen);
return VOLSERREAD_DUMPERROR;
}
code = DumpFile(iodp, vnodeNumber, fdP);
FDH_CLOSE(fdP);
IH_RELEASE(ihP);