diff options
| author | Dan Carpenter <[email protected]> | 2010-05-24 21:33:21 +0000 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2010-05-25 15:07:05 +0000 |
| commit | ea46c8f774f295c45fac48101d54be347d3d453b (patch) | |
| tree | 6a0bc2476e82be68b66cad2b77075e4f36beb1ea /lib/dynamic_debug.c | |
| parent | scripts/get_maintainer.pl: default to not include unspecified tags (diff) | |
| download | kernel-ea46c8f774f295c45fac48101d54be347d3d453b.tar.gz kernel-ea46c8f774f295c45fac48101d54be347d3d453b.zip | |
dynamic_debug: small cleanup in ddebug_proc_write()
This doesn't change behavior at all. In the original code, if nwords was
zero then ddebug_parse_query() would return -EINVAL, now we just do it
earlier.
Signed-off-by: Dan Carpenter <[email protected]>
Acked-by: Jason Baron <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'lib/dynamic_debug.c')
| -rw-r--r-- | lib/dynamic_debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index d6b8b9b1abfe..3df8eb17a607 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_debug.c @@ -456,7 +456,7 @@ static ssize_t ddebug_proc_write(struct file *file, const char __user *ubuf, __func__, (int)len); nwords = ddebug_tokenize(tmpbuf, words, MAXWORDS); - if (nwords < 0) + if (nwords <= 0) return -EINVAL; if (ddebug_parse_query(words, nwords-1, &query)) return -EINVAL; |