path: root/results/classifier/zero-shot/015/all/92957605

permissions: 0.996
semantic: 0.995
debug: 0.994
performance: 0.994
arm: 0.994
alpha: 0.994
PID: 0.993
device: 0.993
socket: 0.993
architecture: 0.992
assembly: 0.992
boot: 0.992
hypervisor: 0.991
virtual: 0.990
network: 0.989
VMM: 0.989
peripherals: 0.989
register: 0.989
operating system: 0.988
risc-v: 0.988
graphic: 0.986
files: 0.986
user-level: 0.985
kernel: 0.983
KVM: 0.982
vnc: 0.981
ppc: 0.981
TCG: 0.974
mistranslation: 0.974
x86: 0.968
i386: 0.958

[Qemu-devel] Fwd:  [BUG] Failed to compile using gcc7.1

Hi all,
I encountered the same problem on gcc 7.1.1 and found Qu's mail in
this list from google search.

Temporarily fix it by specifying the string length in snprintf
directive. Hope this is helpful to other people encountered the same
problem.

@@ -1,9 +1,7 @@
---
--- a/block/blkdebug.c
-                 "blkdebug:%s:%s", s->config_file ?: "",
--- a/block/blkverify.c
-                 "blkverify:%s:%s",
--- a/hw/usb/bus.c
-        snprintf(downstream->path, sizeof(downstream->path), "%s.%d",
-        snprintf(downstream->path, sizeof(downstream->path), "%d", portnr);
--
+++ b/block/blkdebug.c
+                 "blkdebug:%.2037s:%.2037s", s->config_file ?: "",
+++ b/block/blkverify.c
+                 "blkverify:%.2038s:%.2038s",
+++ b/hw/usb/bus.c
+        snprintf(downstream->path, sizeof(downstream->path), "%.12s.%d",
+        snprintf(downstream->path, sizeof(downstream->path), "%.12d", portnr);

Tsung-en Hsiao

>
Qu Wenruo Wrote:
>
>
Hi all,
>
>
After upgrading gcc from 6.3.1 to 7.1.1, qemu can't be compiled with gcc.
>
>
The error is:
>
>
------
>
CC      block/blkdebug.o
>
block/blkdebug.c: In function 'blkdebug_refresh_filename':
>
>
block/blkdebug.c:693:31: error: '%s' directive output may be truncated
>
writing up to 4095 bytes into a region of size 4086
>
[-Werror=format-truncation=]
>
>
"blkdebug:%s:%s", s->config_file ?: "",
>
^~
>
In file included from /usr/include/stdio.h:939:0,
>
from /home/adam/qemu/include/qemu/osdep.h:68,
>
from block/blkdebug.c:25:
>
>
/usr/include/bits/stdio2.h:64:10: note: '__builtin___snprintf_chk' output 11
>
or more bytes (assuming 4106) into a destination of size 4096
>
>
return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1,
>
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
__bos (__s), __fmt, __va_arg_pack ());
>
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
cc1: all warnings being treated as errors
>
make: *** [/home/adam/qemu/rules.mak:69: block/blkdebug.o] Error 1
>
------
>
>
It seems that gcc 7 is introducing more restrict check for printf.
>
>
If using clang, although there are some extra warning, it can at least pass
>
the compile.
>
>
Thanks,
>
Qu

Hi Tsung-en,

On 06/11/2017 04:08 PM, Tsung-en Hsiao wrote:
Hi all,
I encountered the same problem on gcc 7.1.1 and found Qu's mail in
this list from google search.

Temporarily fix it by specifying the string length in snprintf
directive. Hope this is helpful to other people encountered the same
problem.
Thank your for sharing this.
@@ -1,9 +1,7 @@
---
--- a/block/blkdebug.c
-                 "blkdebug:%s:%s", s->config_file ?: "",
--- a/block/blkverify.c
-                 "blkverify:%s:%s",
--- a/hw/usb/bus.c
-        snprintf(downstream->path, sizeof(downstream->path), "%s.%d",
-        snprintf(downstream->path, sizeof(downstream->path), "%d", portnr);
--
+++ b/block/blkdebug.c
+                 "blkdebug:%.2037s:%.2037s", s->config_file ?: "",
It is a rather funny way to silent this warning :) Truncating the
filename until it fits.
However I don't think it is the correct way since there is indeed an
overflow of bs->exact_filename.
Apparently exact_filename from "block/block_int.h" is defined to hold a
pathname:
char exact_filename[PATH_MAX];
but is used for more than that (for example in blkdebug.c it might use
until 10+2*PATH_MAX chars).
I suppose it started as a buffer to hold a pathname then more block
drivers were added and this buffer ended used differently.
If it is a multi-purpose buffer one safer option might be to declare it
as a GString* and use g_string_printf().
I CC'ed the block folks to have their feedback.

Regards,

Phil.
+++ b/block/blkverify.c
+                 "blkverify:%.2038s:%.2038s",
+++ b/hw/usb/bus.c
+        snprintf(downstream->path, sizeof(downstream->path), "%.12s.%d",
+        snprintf(downstream->path, sizeof(downstream->path), "%.12d", portnr);

Tsung-en Hsiao
Qu Wenruo Wrote:

Hi all,

After upgrading gcc from 6.3.1 to 7.1.1, qemu can't be compiled with gcc.

The error is:

------
 CC      block/blkdebug.o
block/blkdebug.c: In function 'blkdebug_refresh_filename':

block/blkdebug.c:693:31: error: '%s' directive output may be truncated writing 
up to 4095 bytes into a region of size 4086 [-Werror=format-truncation=]

                 "blkdebug:%s:%s", s->config_file ?: "",
                              ^~
In file included from /usr/include/stdio.h:939:0,
                from /home/adam/qemu/include/qemu/osdep.h:68,
                from block/blkdebug.c:25:

/usr/include/bits/stdio2.h:64:10: note: '__builtin___snprintf_chk' output 11 or 
more bytes (assuming 4106) into a destination of size 4096

  return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1,
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       __bos (__s), __fmt, __va_arg_pack ());
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make: *** [/home/adam/qemu/rules.mak:69: block/blkdebug.o] Error 1
------

It seems that gcc 7 is introducing more restrict check for printf.

If using clang, although there are some extra warning, it can at least pass the 
compile.

Thanks,
Qu

On 2017-06-12 05:19, Philippe Mathieu-Daudé wrote:
>
Hi Tsung-en,
>
>
On 06/11/2017 04:08 PM, Tsung-en Hsiao wrote:
>
> Hi all,
>
> I encountered the same problem on gcc 7.1.1 and found Qu's mail in
>
> this list from google search.
>
>
>
> Temporarily fix it by specifying the string length in snprintf
>
> directive. Hope this is helpful to other people encountered the same
>
> problem.
>
>
Thank your for sharing this.
>
>
>
>
> @@ -1,9 +1,7 @@
>
> ---
>
> --- a/block/blkdebug.c
>
> -                 "blkdebug:%s:%s", s->config_file ?: "",
>
> --- a/block/blkverify.c
>
> -                 "blkverify:%s:%s",
>
> --- a/hw/usb/bus.c
>
> -        snprintf(downstream->path, sizeof(downstream->path), "%s.%d",
>
> -        snprintf(downstream->path, sizeof(downstream->path), "%d",
>
> portnr);
>
> --
>
> +++ b/block/blkdebug.c
>
> +                 "blkdebug:%.2037s:%.2037s", s->config_file ?: "",
>
>
It is a rather funny way to silent this warning :) Truncating the
>
filename until it fits.
>
>
However I don't think it is the correct way since there is indeed an
>
overflow of bs->exact_filename.
>
>
Apparently exact_filename from "block/block_int.h" is defined to hold a
>
pathname:
>
char exact_filename[PATH_MAX];
>
>
but is used for more than that (for example in blkdebug.c it might use
>
until 10+2*PATH_MAX chars).
In any case, truncating the filenames will do just as much as truncating
the result: You'll get an unusable filename.

>
I suppose it started as a buffer to hold a pathname then more block
>
drivers were added and this buffer ended used differently.
>
>
If it is a multi-purpose buffer one safer option might be to declare it
>
as a GString* and use g_string_printf().
What it is supposed to be now is just an information string we can print
to the user, because strings are nicer than JSON objects. There are some
commands that take a filename for identifying a block node, but I dream
we can get rid of them in 3.0...

The right solution is to remove it altogether and have a
"char *bdrv_filename(BlockDriverState *bs)" function (which generates
the filename every time it's called). I've been working on this for some
years now, actually, but it was never pressing enough to get it finished
(so I never had enough time).

What we can do in the meantime is to not generate a plain filename if it
won't fit into bs->exact_filename.

(The easiest way to do this probably would be to truncate
bs->exact_filename back to an empty string if snprintf() returns a value
greater than or equal to the length of bs->exact_filename.)

What to do about hw/usb/bus.c I don't know (I guess the best solution
would be to ignore the warning, but I don't suppose that is going to work).

Max

>
>
I CC'ed the block folks to have their feedback.
>
>
Regards,
>
>
Phil.
>
>
> +++ b/block/blkverify.c
>
> +                 "blkverify:%.2038s:%.2038s",
>
> +++ b/hw/usb/bus.c
>
> +        snprintf(downstream->path, sizeof(downstream->path), "%.12s.%d",
>
> +        snprintf(downstream->path, sizeof(downstream->path), "%.12d",
>
> portnr);
>
>
>
> Tsung-en Hsiao
>
>
>
>> Qu Wenruo Wrote:
>
>>
>
>> Hi all,
>
>>
>
>> After upgrading gcc from 6.3.1 to 7.1.1, qemu can't be compiled with
>
>> gcc.
>
>>
>
>> The error is:
>
>>
>
>> ------
>
>>  CC      block/blkdebug.o
>
>> block/blkdebug.c: In function 'blkdebug_refresh_filename':
>
>>
>
>> block/blkdebug.c:693:31: error: '%s' directive output may be
>
>> truncated writing up to 4095 bytes into a region of size 4086
>
>> [-Werror=format-truncation=]
>
>>
>
>>                  "blkdebug:%s:%s", s->config_file ?: "",
>
>>                               ^~
>
>> In file included from /usr/include/stdio.h:939:0,
>
>>                 from /home/adam/qemu/include/qemu/osdep.h:68,
>
>>                 from block/blkdebug.c:25:
>
>>
>
>> /usr/include/bits/stdio2.h:64:10: note: '__builtin___snprintf_chk'
>
>> output 11 or more bytes (assuming 4106) into a destination of size 4096
>
>>
>
>>   return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1,
>
>>          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
>>        __bos (__s), __fmt, __va_arg_pack ());
>
>>        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
>> cc1: all warnings being treated as errors
>
>> make: *** [/home/adam/qemu/rules.mak:69: block/blkdebug.o] Error 1
>
>> ------
>
>>
>
>> It seems that gcc 7 is introducing more restrict check for printf.
>
>>
>
>> If using clang, although there are some extra warning, it can at
>
>> least pass the compile.
>
>>
>
>> Thanks,
>
>> Qu
>
>
signature.asc
Description:
OpenPGP digital signature