git:4.2:ec6d3969: xmlrpc: remove redeclaration of snprintf and vsnprintf - sr-dev

8 Jun 2016

Module: kamailio
Branch: 4.2
Commit: ec6d396911d48c5300a38935cf6efca20602aeeb
URL: https://github.com/kamailio/kamailio/commit/ec6d396911d48c5300a38935cf6efca…

Author: Victor Seva &lt;linuxmaniac(a)torreviejawireless.org&gt;
Committer: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt;
Date: 2016-06-08T09:34:34+02:00

xmlrpc: remove redeclaration of snprintf and vsnprintf

compilation was failling with clang:

...
  CC (clang) [M xmlrpc.so]		xmlrpc.o
 xmlrpc.c:152:5: error: expected parameter declarator
 int snprintf(char *str, size_t size, const char *format, ...);
     ^
 /usr/include/x86_64-linux-gnu/bits/stdio2.h:69:39: note: expanded from macro
'snprintf'
   __builtin___snprintf_chk (str, len, __USE_FORTIFY_LEVEL - 1, __bos (str), \
                                       ^
 /usr/include/features.h:329:31: note: expanded from macro '__USE_FORTIFY_LEVEL'
 #  define __USE_FORTIFY_LEVEL 2
                               ^
 xmlrpc.c:152:5: error: expected ')'
 /usr/include/x86_64-linux-gnu/bits/stdio2.h:69:39: note: expanded from macro
'snprintf'
   __builtin___snprintf_chk (str, len, __USE_FORTIFY_LEVEL - 1, __bos (str), \
                                       ^
 /usr/include/features.h:329:31: note: expanded from macro '__USE_FORTIFY_LEVEL'
 #  define __USE_FORTIFY_LEVEL 2
                               ^
 xmlrpc.c:152:5: note: to match this '('
 /usr/include/x86_64-linux-gnu/bits/stdio2.h:69:28: note: expanded from macro
'snprintf'
   __builtin___snprintf_chk (str, len, __USE_FORTIFY_LEVEL - 1, __bos (str), \
                            ^
 xmlrpc.c:152:5: error: conflicting types for '__builtin___snprintf_chk'
 int snprintf(char *str, size_t size, const char *format, ...);
     ^
 /usr/include/x86_64-linux-gnu/bits/stdio2.h:69:3: note: expanded from macro
'snprintf'
   __builtin___snprintf_chk (str, len, __USE_FORTIFY_LEVEL - 1, __bos (str), \
   ^
 xmlrpc.c:152:5: note: '__builtin___snprintf_chk' is a builtin with type 'int
(char *, unsigned long, int, unsigned long, const char *, ...)'
 /usr/include/x86_64-linux-gnu/bits/stdio2.h:69:3: note: expanded from macro
'snprintf'
   __builtin___snprintf_chk (str, len, __USE_FORTIFY_LEVEL - 1, __bos (str), \
   ^
 3 errors generated. 
(cherry picked from commit 358de10a94bef4df6b1811ac8e88ea4e7919542c)
(cherry picked from commit efe5903cd17de9c5d94610e2345254e347e3aeb0)

---

Modified: modules/xmlrpc/xmlrpc.c

---

Diff: 
https://github.com/kamailio/kamailio/commit/ec6d396911d48c5300a38935cf6efca…
Patch:
https://github.com/kamailio/kamailio/commit/ec6d396911d48c5300a38935cf6efca…

---

diff --git a/modules/xmlrpc/xmlrpc.c b/modules/xmlrpc/xmlrpc.c
index e97a3f3..6743a30 100644
--- a/modules/xmlrpc/xmlrpc.c
+++ b/modules/xmlrpc/xmlrpc.c
@@ -149,13 +149,6 @@
 
 MODULE_VERSION
 
-#if defined (__OS_darwin) || defined (__OS_freebsd)
-/* redeclaration of functions from stdio.h throws errors */
-#else
-int snprintf(char *str, size_t size, const char *format, ...);
-int vsnprintf(char *str, size_t size, const char *format, va_list ap);
-#endif
-
 static int process_xmlrpc(sip_msg_t* msg);
 static int dispatch_rpc(sip_msg_t* msg, char* s1, char* s2);
 static int xmlrpc_reply(sip_msg_t* msg, char* code, char* reason);



    