From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Sun, 10 Apr 2022 20:19:21 +0000 (-0500)
Subject: Don't print file loacations if this isn't a debug build
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b5ce85149ed87536eabcb433e837dfc76447314c;p=thirdparty%2Ffreeradius-server.git

Don't print file loacations if this isn't a debug build
---

diff --git a/src/lib/util/atexit.c b/src/lib/util/atexit.c
index b4c8fb0128..1e8def741e 100644
--- a/src/lib/util/atexit.c
+++ b/src/lib/util/atexit.c
@@ -396,9 +396,16 @@ int fr_atexit_thread_trigger_all(void)
 			to_free = ee;
 			ee = fr_dlist_remove(&list->head, ee);
 			if (talloc_free(to_free) < 0) {
-				fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p (alloced %s:%u)",
+				fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p"
+#ifndef NDEBUG
+							" (alloced %s:%u)",
+#endif
 							list, to_free,
-							to_free->func, to_free->uctx, to_free->file, to_free->line);
+							to_free->func, to_free->uctx
+#ifndef NDEBUG
+							, to_free->file, to_free->line
+#endif
+							);
 				return -1;
 			}
 		}
@@ -495,9 +502,16 @@ int fr_atexit_global_trigger_all(void)
 		to_free = e;
 		e = fr_dlist_remove(&fr_atexit_global->head, e);
 		if (talloc_free(to_free) < 0) {
-			fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p (alloced %s:%u)",
+			fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p"
+#ifndef NDEBUG
+						" (alloced %s:%u)",
+#endif
 						fr_atexit_global, to_free,
-						to_free->func, to_free->uctx, to_free->file, to_free->line);
+						to_free->func, to_free->uctx
+#ifndef NDEBUG
+						, to_free->file, to_free->line
+#endif
+						);
 			return -1;
 		}
 	}
@@ -541,9 +555,16 @@ int fr_atexit_trigger(bool uctx_scope, fr_atexit_t func, void const *uctx)
 		to_free = e;
 		e = fr_dlist_remove(&fr_atexit_global->head, e);
 		if (talloc_free(to_free) < 0) {
-			fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p (alloced %s:%u)",
+			fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p"
+#ifndef NDEBUG
+						" (alloced %s:%u)",
+#endif
 						fr_atexit_global, to_free,
-						to_free->func, to_free->uctx, to_free->file, to_free->line);
+						to_free->func, to_free->uctx
+#ifndef NDEBUG
+						, to_free->file, to_free->line
+#endif
+						);
 			return -1;
 		}
 	}
@@ -574,9 +595,16 @@ do_threads:
 			to_free = ee;
 			ee = fr_dlist_remove(&list->head, ee);
 			if (talloc_free(to_free) < 0) {
-				fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p (alloced %s:%u)",
+				fr_strerror_printf_push("atexit handler failed %p/%p func=%p, uctx=%p"
+#ifndef NDEBUG
+							" (alloced %s:%u)",
+#endif
 							list, to_free,
-							to_free->func, to_free->uctx, to_free->file, to_free->line);
+							to_free->func, to_free->uctx
+#ifndef NDEBUG
+							, to_free->file, to_free->line
+#endif
+							);
 				return -1;
 			}
 		}