r304026 added code that started statistics gathering for an operation

before the operation number (the variable called "op") was sanity checked.
This patch moves the code down to below the range sanity check for "op".
This commit is contained in:
Rick Macklem 2018-11-20 01:52:45 +00:00
parent 3d62bdba63
commit 1d171e7971

View File

@ -766,11 +766,6 @@ nfsrvd_compound(struct nfsrv_descript *nd, int isdgram, u_char *tag,
*repp = *tl;
op = fxdr_unsigned(int, *tl);
NFSD_DEBUG(4, "op=%d\n", op);
binuptime(&start_time);
nfsrvd_statstart(op, &start_time);
statsinprog = 1;
if (op < NFSV4OP_ACCESS ||
(op >= NFSV4OP_NOPS && (nd->nd_flag & ND_NFSV41) == 0) ||
(op >= NFSV41_NOPS && (nd->nd_flag & ND_NFSV41) != 0)) {
@ -782,6 +777,11 @@ nfsrvd_compound(struct nfsrv_descript *nd, int isdgram, u_char *tag,
} else {
repp++;
}
binuptime(&start_time);
nfsrvd_statstart(op, &start_time);
statsinprog = 1;
if (i == 0)
op0 = op;
if (i == numops - 1)