Explain why snprintf has to be overridden in this way.

(This used to be commit a4cc69d9459879c3fa5e782efd643d4e8cd334ba)

1 files changed, 8 insertions, 1 deletions

diff --git a/source3/lib/snprintf.c b/source3/lib/snprintf.c
index 6e7613a276..7d6da95f8e 100644
--- a/source3/lib/snprintf.c
+++ b/source3/lib/snprintf.c
@@ -757,7 +757,14 @@ static void dopr_outch(char *buffer, size_t *currlen, size_t maxlen, char c)
 }
 #endif
 
-/* yes this really must be a ||. Don't muck with this (tridge) */
+/* yes this really must be a ||. Don't muck wiith this (tridge)
+ *
+ * The logic for these two is that we need our own definition if the
+ * OS *either* has no definition of *sprintf, or if it does have one
+ * that doesn't work properly according to the autoconf test.  Perhaps
+ * these should really be smb_snprintf to avoid conflicts with buggy
+ * linkers? -- mbp
+ */
 #if !defined(HAVE_SNPRINTF) || !defined(HAVE_C99_SNPRINTF)
  int snprintf(char *str,size_t count,const char *fmt,...)
 {