diff options
author | Keir Fraser <keir.fraser@citrix.com> | 2007-12-28 15:23:28 +0000 |
---|---|---|
committer | Keir Fraser <keir.fraser@citrix.com> | 2007-12-28 15:23:28 +0000 |
commit | 8f8a3b56592acf31ffcbf6b6747e247dd53da355 (patch) | |
tree | 6b7fdf56e3ede7cdb83d04371171a34b1fc2de37 /extras/mini-os/lib/printf.c | |
parent | f53513b012968240ba490efe873982d213a85d46 (diff) | |
download | xen-8f8a3b56592acf31ffcbf6b6747e247dd53da355.tar.gz xen-8f8a3b56592acf31ffcbf6b6747e247dd53da355.tar.bz2 xen-8f8a3b56592acf31ffcbf6b6747e247dd53da355.zip |
In C99 (7.19.6.2.10), the behavior of scanf("stuff... %n", ..., &n) is
said to be undefined if the value to be put is n can't fit. I guess
we can safely consider that the same applies to printf.
Signed-off-by: Samuel Thibault <samuel.thibault@citrix.com>
Diffstat (limited to 'extras/mini-os/lib/printf.c')
-rw-r--r-- | extras/mini-os/lib/printf.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/extras/mini-os/lib/printf.c b/extras/mini-os/lib/printf.c index 676f4e74a1..a6767e48ad 100644 --- a/extras/mini-os/lib/printf.c +++ b/extras/mini-os/lib/printf.c @@ -426,8 +426,6 @@ int vsnprintf(char *buf, size_t size, const char *fmt, va_list args) case 'n': - /* FIXME: - * What does C99 say about the overflow case here? */ if (qualifier == 'l') { long * ip = va_arg(args, long *); *ip = (str - buf); @@ -446,7 +444,7 @@ int vsnprintf(char *buf, size_t size, const char *fmt, va_list args) ++str; continue; - /* integer number formats - set up the flags and "break" */ + /* integer number formats - set up the flags and "break" */ case 'o': base = 8; break; |