aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Pluzhnikov <ppluzhnikov@google.com>2023-03-25 21:27:01 +0000
committerPaul Pluzhnikov <ppluzhnikov@google.com>2023-03-26 00:22:30 +0000
commit1b5e65ef6a442fdccf88d43c3048f98292d85631 (patch)
tree947afa2285e76b176f18ee8a2e8bcb7b3c50deba
parentdb9b47e9f996bbdb831580ff7343542a017c80ee (diff)
downloadglibc-1b5e65ef6a442fdccf88d43c3048f98292d85631.zip
glibc-1b5e65ef6a442fdccf88d43c3048f98292d85631.tar.gz
glibc-1b5e65ef6a442fdccf88d43c3048f98292d85631.tar.bz2
Minor: don't call _dl_debug_update (which can have side effects) inside assert
-rw-r--r--elf/dl-open.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/elf/dl-open.c b/elf/dl-open.c
index 91a2d8a..2d985e2 100644
--- a/elf/dl-open.c
+++ b/elf/dl-open.c
@@ -578,7 +578,9 @@ dl_open_worker_begin (void *a)
if ((mode & RTLD_GLOBAL) && new->l_global == 0)
add_to_global_update (new);
- assert (_dl_debug_update (args->nsid)->r_state == RT_CONSISTENT);
+ const int r_state __attribute__ ((unused))
+ = _dl_debug_update (args->nsid)->r_state;
+ assert (r_state == RT_CONSISTENT);
return;
}
@@ -927,7 +929,9 @@ no more namespaces available for dlmopen()"));
_dl_signal_exception (errcode, &exception, NULL);
}
- assert (_dl_debug_update (args.nsid)->r_state == RT_CONSISTENT);
+ const int r_state __attribute__ ((unused))
+ = _dl_debug_update (args.nsid)->r_state;
+ assert (r_state == RT_CONSISTENT);
/* Release the lock. */
__rtld_lock_unlock_recursive (GL(dl_load_lock));