[Teem-commit] SF.net SVN: teem:[7536] teem/trunk/src/nrrd/kernel. c

[<kin...@us...>]

Revision: 7536
          http://sourceforge.net/p/teem/code/7536
Author:   kindlmann
Date:     2025-09-30 14:33:56 +0000 (Tue, 30 Sep 2025)
Log Message:
-----------
reducing warnings associated with sprintf and snprintf

Modified Paths:
--------------
    teem/trunk/src/nrrd/kernel.c

Modified: teem/trunk/src/nrrd/kernel.c
===================================================================
--- teem/trunk/src/nrrd/kernel.c	2025-09-30 07:29:51 UTC (rev 7535)
+++ teem/trunk/src/nrrd/kernel.c	2025-09-30 14:33:56 UTC (rev 7536)
@@ -3354,7 +3354,8 @@
 nrrdKernelSpecCompare(const NrrdKernelSpec *aa, const NrrdKernelSpec *bb, int *differ,
                       char explain[AIR_STRLEN_LARGE + 1]) {
   static const char me[] = "nrrdKernelSpecCompare";
-  char subexplain[AIR_STRLEN_LARGE + 1];
+#define ASL AIR_STRLEN_LARGE
+  char subexplain[ASL + 1];
 
   if (!(differ)) {
     biffAddf(NRRD, "%s: got NULL differ", me);
@@ -3362,8 +3363,8 @@
   }
   if (!!aa != !!bb) {
     if (explain) {
-      sprintf(explain, "different NULL-ities of kspec itself %s != %s",
-              aa ? "non-NULL" : "NULL", bb ? "non-NULL" : "NULL");
+      snprintf(explain, ASL + 1, "different NULL-ities of kspec itself %s != %s",
+               aa ? "non-NULL" : "NULL", bb ? "non-NULL" : "NULL");
     }
     *differ = 1;
     return 0;
@@ -3375,8 +3376,8 @@
   }
   if (!!aa->kernel != !!bb->kernel) {
     if (explain) {
-      sprintf(explain, "different NULL-ities of kspec->kernel %s != %s",
-              aa->kernel ? "non-NULL" : "NULL", bb->kernel ? "non-NULL" : "NULL");
+      snprintf(explain, ASL + 1, "different NULL-ities of kspec->kernel %s != %s",
+               aa->kernel ? "non-NULL" : "NULL", bb->kernel ? "non-NULL" : "NULL");
     }
     *differ = 1;
     return 0;
@@ -3393,12 +3394,14 @@
   }
   if (*differ) {
     if (explain) {
-      sprintf(explain, "kern/parm pairs differ: %s", subexplain);
+      snprintf(explain, ASL + 1, "kern/parm pairs differ: %s",
+               airStrunc(subexplain, ASL + 1, 30));
     }
     *differ = 1; /* losing ordering info (of dubious value) */
     return 0;
   }
   *differ = 0;
+#undef ASL
   return 0;
 }
 

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.