diff options
author | Eric Van Hensbergen <ericvh@gmail.com> | 2008-10-17 13:45:23 -0400 |
---|---|---|
committer | Eric Van Hensbergen <ericvh@gmail.com> | 2008-10-17 13:45:23 -0400 |
commit | f0a0ac2ee50c62cf4ad9b06cf8a12435cc5ac44d (patch) | |
tree | 6d1b42f61f3a7fc069d100b73579e42418c1ec8a /net/9p/protocol.c | |
parent | 57c7b4e68edf3b4fe7f977db9ad437e0f7f7c382 (diff) |
9p: fix oops in protocol stat parsing error path.
When we get an error on parsing a stat due to a protocol bug,
we can generate an oops during cleanup because we didn't
initialize the string pointers in the stat structure.
Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
Diffstat (limited to 'net/9p/protocol.c')
-rw-r--r-- | net/9p/protocol.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/9p/protocol.c b/net/9p/protocol.c index 84fa21271876..29be52439086 100644 --- a/net/9p/protocol.c +++ b/net/9p/protocol.c | |||
@@ -215,9 +215,9 @@ p9pdu_vreadf(struct p9_fcall *pdu, int optional, const char *fmt, va_list ap) | |||
215 | struct p9_wstat *stbuf = | 215 | struct p9_wstat *stbuf = |
216 | va_arg(ap, struct p9_wstat *); | 216 | va_arg(ap, struct p9_wstat *); |
217 | 217 | ||
218 | stbuf->extension = NULL; | 218 | memset(stbuf, 0, sizeof(struct p9_wstat)); |
219 | stbuf->n_uid = stbuf->n_gid = stbuf->n_muid = | 219 | stbuf->n_uid = stbuf->n_gid = stbuf->n_muid = |
220 | -1; | 220 | -1; |
221 | errcode = | 221 | errcode = |
222 | p9pdu_readf(pdu, optional, | 222 | p9pdu_readf(pdu, optional, |
223 | "wwdQdddqssss?sddd", | 223 | "wwdQdddqssss?sddd", |