Merge from HEAD - add PRINTF_ATTRIBUTE to a few more functions.

(This used to be commit 9e5297131cc53d7161aa74566f147b98e1c27aaa)

diff --git a/source3/include/includes.h b/source3/include/includes.h
index 58f1565d80a9883bb60148b46e2ee7074315b432..391d1bf1f919005dd69d75f29c57554e4d7ec1dc 100644 (file)
--- a/source3/include/includes.h
+++ b/source3/include/includes.h
@@ -1156,6 +1156,15 @@ int snprintf(char *,size_t ,const char *, ...) PRINTF_ATTRIBUTE(3,4);
 int asprintf(char **,const char *, ...) PRINTF_ATTRIBUTE(2,3);
 #endif
 
+void sys_adminlog(int priority, const char *format_str, ...) PRINTF_ATTRIBUTE(2,3);
+
+int pstr_sprintf(pstring s, const char *fmt, ...) PRINTF_ATTRIBUTE(2,3);
+int fstr_sprintf(fstring s, const char *fmt, ...) PRINTF_ATTRIBUTE(2,3);
+
+int d_vfprintf(FILE *f, const char *format, va_list ap) PRINTF_ATTRIBUTE(2,0);
+
+int smb_xvasprintf(char **ptr, const char *format, va_list ap) PRINTF_ATTRIBUTE(2,0);
+
 /* we used to use these fns, but now we have good replacements
    for snprintf and vsnprintf */
 #define slprintf snprintf

diff --git a/source3/include/xfile.h b/source3/include/xfile.h
index a573b59a4a7da6d11f794caefbb4277f84fb5866..89fa9d1e1188d81e806010766d41b475427f7f32 100644 (file)
--- a/source3/include/xfile.h
+++ b/source3/include/xfile.h
@@ -44,4 +44,6 @@ extern XFILE *x_stdin, *x_stdout, *x_stderr;
 
 #define x_getc(f) x_fgetc(f)
 
+int x_vfprintf(XFILE *f, const char *format, va_list ap) PRINTF_ATTRIBUTE(2, 0);
+int x_fprintf(XFILE *f, const char *format, ...) PRINTF_ATTRIBUTE(2, 3);
 #endif /* _XFILE_H_ */

diff --git a/source3/lib/dprintf.c b/source3/lib/dprintf.c
index f0f09e199d468bef70adad853034f65703d88e34..70387bbd61823248a381a8c0d96b51e403c395b6 100644 (file)
--- a/source3/lib/dprintf.c
+++ b/source3/lib/dprintf.c
@@ -31,7 +31,7 @@
 
 #include "includes.h"
 
-int d_vfprintf(FILE *f, const char *format, va_list ap)
+ int d_vfprintf(FILE *f, const char *format, va_list ap)
 {
        char *p, *p2;
        int ret, maxlen, clen;

diff --git a/source3/lib/system.c b/source3/lib/system.c
index 873b8737d506399d7e36d46df7351329947275a7..18cccda935838d401488c828cbfc45ef7fcf1bb6 100644 (file)
--- a/source3/lib/system.c
+++ b/source3/lib/system.c
@@ -1233,7 +1233,7 @@ int sys_dup2(int oldfd, int newfd)
  Wrapper for Admin Logs.
 ****************************************************************************/
 
-void sys_adminlog(int priority, char *format_str, ...)
+ void sys_adminlog(int priority, const char *format_str, ...) 
 {
        va_list ap;
        int ret;

diff --git a/source3/lib/util.c b/source3/lib/util.c
index 0adfe34032ddda0aed20f965719d49567e1f1f45..b0714dcbebe69f405637664a906e1f6f19249478 100644 (file)
--- a/source3/lib/util.c
+++ b/source3/lib/util.c
@@ -2080,7 +2080,7 @@ char *smb_xstrndup(const char *s, size_t n)
   vasprintf that aborts on malloc fail
 */
 
-int smb_xvasprintf(char **ptr, const char *format, va_list ap)
+ int smb_xvasprintf(char **ptr, const char *format, va_list ap)
 {
        int n;
        va_list ap2;

diff --git a/source3/lib/util_str.c b/source3/lib/util_str.c
index f3fa89b05f37554382c9998ecbc8661f5bbc6b4e..5a1f159bdbf6ce004bd62260b70de81062d3224c 100644 (file)
--- a/source3/lib/util_str.c
+++ b/source3/lib/util_str.c
@@ -1116,7 +1116,7 @@ char *binary_string(char *buf, int len)
  Just a typesafety wrapper for snprintf into a pstring.
 ********************************************************************/
 
-int pstr_sprintf(pstring s, const char *fmt, ...)
+ int pstr_sprintf(pstring s, const char *fmt, ...)
 {
        va_list ap;
        int ret;
@@ -1131,7 +1131,7 @@ int pstr_sprintf(pstring s, const char *fmt, ...)
  Just a typesafety wrapper for snprintf into a fstring.
 ********************************************************************/
 
-int fstr_sprintf(fstring s, const char *fmt, ...)
+ int fstr_sprintf(fstring s, const char *fmt, ...)
 {
        va_list ap;
        int ret;

diff --git a/source3/lib/xfile.c b/source3/lib/xfile.c
index 7621712e9ad31192f93d97917b7ec61b82de306b..57f3e2763898fa16725c8efe0bc714b977330b85 100644 (file)
--- a/source3/lib/xfile.c
+++ b/source3/lib/xfile.c
@@ -184,7 +184,7 @@ int x_fwrite(const void *p, size_t size, size_t nmemb, XFILE *f)
 }
 
 /* thank goodness for asprintf() */
-int x_vfprintf(XFILE *f, const char *format, va_list ap)
+ int x_vfprintf(XFILE *f, const char *format, va_list ap)
 {
        char *p;
        int len, ret;
@@ -199,7 +199,7 @@ int x_vfprintf(XFILE *f, const char *format, va_list ap)
        return ret;
 }
 
-int x_fprintf(XFILE *f, const char *format, ...)
+ int x_fprintf(XFILE *f, const char *format, ...)
 {
        va_list ap;
        int ret;