HACKING: Update status of format checking
PATCH 1/1 HACKING: Update status of format checking
This patch was already sent on 2011-01-24:
Hopefully all functions with printf like arguments now use format checking.
This was tested with default build configuration on linux and windows hosts (including some cross compilations), so chances are good that there remain few (if any) functions without format checking.
Therefore the last comment in HACKING is no longer valid but misleading.
--- HACKING | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-)
HACKING +0 -3
@@ -120,6 +120,3 @@ gcc's printf attribute directive in the prototype. This makes it so gcc's -Wformat and -Wformat-security options can do their jobs and cross-check format strings with the number and types of arguments. - -Currently many functions in QEMU are not following this rule but -patches to add the attribute would be very much appreciated.
Signature · 1 line
1.7.2.3
Anthony Liguori
AppliedQuoted 9 lines · Stefan Weil
This patch was already sent on 2011-01-24:
Hopefully all functions with printf like arguments now use format checking.
This was tested with default build configuration on linux and windows hosts (including some cross compilations), so chances are good that there remain few (if any) functions without format checking.
Quoted 4 lines
Therefore the last comment in HACKING is no longer valid but misleading.
Cc Blue Swirl <[email protected]>Signed-off-by Stefan Weil <[email protected]>
Applied. Thanks.
Regards,
Anthony Liguori
Quoted 15 lines
--- HACKING | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-)HACKING +0
-3@@ -120,6 +120,3 @@ gcc's printf attribute directive in the prototype. This makes it so gcc's -Wformat and -Wformat-security options can do their jobs and cross-check format strings with the number and types of arguments. - -Currently many functions in QEMU are not following this rule but -patches to add the attribute would be very much appreciated.