diff options
| author | Florian Weimer <fweimer@redhat.com> | 2025-09-12 21:33:34 +0200 |
|---|---|---|
| committer | Florian Weimer <fweimer@redhat.com> | 2025-09-19 09:20:48 +0200 |
| commit | cf926cd7fb9500035be19bca90ba9c855dcfcf92 (patch) | |
| tree | 5d71f8f3247e26f9d7070649eea062ed820fdbb6 | |
| parent | 9867e44cdca3a1ddd39b67239aeecc80d0226eda (diff) | |
| download | glibc-release/2.41/master.zip glibc-release/2.41/master.tar.gz glibc-release/2.41/master.tar.bz2 | |
nss: Group merge does not react to ERANGE during merge (bug 33361)release/2.41/master
The break statement in CHECK_MERGE is expected to exit the surrounding
while loop, not the do-while loop with in the macro. Remove the
do-while loop from the macro. It is not needed to turn the macro
expansion into a single statement due to the way CHECK_MERGE is used
(and the statement expression would cover this anyway).
Reviewed-by: Collin Funk <collin.funk1@gmail.com>
(cherry picked from commit 0fceed254559836b57ee05188deac649bc505d05)
| -rw-r--r-- | NEWS | 1 | ||||
| -rw-r--r-- | nss/getXXbyYY_r.c | 18 |
2 files changed, 8 insertions, 11 deletions
@@ -41,6 +41,7 @@ The following bugs were resolved with this release: [33185] Fix double-free after allocation failure in regcomp [33234] Use TLS initial-exec model for __libc_tsd_CTYPE_* thread variables [33245] nptl: nptl: error in internal cancellation syscall handling + [33361] nss: Group merge does not react to ERANGE during merge Version 2.41 diff --git a/nss/getXXbyYY_r.c b/nss/getXXbyYY_r.c index eae6c34..2b0735f 100644 --- a/nss/getXXbyYY_r.c +++ b/nss/getXXbyYY_r.c @@ -157,19 +157,15 @@ __merge_einval (LOOKUP_TYPE *a, #define CHECK_MERGE(err, status) \ ({ \ - do \ + if (err) \ { \ - if (err) \ - { \ - __set_errno (err); \ - if (err == ERANGE) \ - status = NSS_STATUS_TRYAGAIN; \ - else \ - status = NSS_STATUS_UNAVAIL; \ - break; \ - } \ + __set_errno (err); \ + if (err == ERANGE) \ + status = NSS_STATUS_TRYAGAIN; \ + else \ + status = NSS_STATUS_UNAVAIL; \ + break; \ } \ - while (0); \ }) /* Type of the lookup function we need here. */ |