diff options
author | Jan Beulich <jbeulich@suse.com> | 2011-09-07 10:36:55 +0100 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2011-09-07 10:36:55 +0100 |
commit | cd4ab341d27ee080ccad1cf85a366606a98dbc49 (patch) | |
tree | 196b62280dd6924cd9acd0291f4216045e3aa48a /xen/common/bitmap.c | |
parent | 090a5436289faaae79b1f49b29f4eeefc89d017e (diff) | |
download | xen-cd4ab341d27ee080ccad1cf85a366606a98dbc49.tar.gz xen-cd4ab341d27ee080ccad1cf85a366606a98dbc49.tar.bz2 xen-cd4ab341d27ee080ccad1cf85a366606a98dbc49.zip |
bitmap_scnlistprintf() should always zero-terminate its output buffer
... as long as it has non-zero size. So far this would not happen if
the passed in CPU mask was empty.
Also fix the comment describing the return value to actually match
reality.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Diffstat (limited to 'xen/common/bitmap.c')
-rw-r--r-- | xen/common/bitmap.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/xen/common/bitmap.c b/xen/common/bitmap.c index 363572bedb..a34f519359 100644 --- a/xen/common/bitmap.c +++ b/xen/common/bitmap.c @@ -363,9 +363,8 @@ static inline int bscnl_emit(char *buf, int buflen, int rbot, int rtop, int len) * the range. Output format is compatible with the format * accepted as input by bitmap_parselist(). * - * The return value is the number of characters which would be - * generated for the given input, excluding the trailing '\0', as - * per ISO C99. + * The return value is the number of characters which were output, + * excluding the trailing '\0'. */ int bitmap_scnlistprintf(char *buf, unsigned int buflen, const unsigned long *maskp, int nmaskbits) @@ -383,6 +382,8 @@ int bitmap_scnlistprintf(char *buf, unsigned int buflen, rbot = cur; } } + if (!len && buflen) + *buf = 0; return len; } EXPORT_SYMBOL(bitmap_scnlistprintf); |