AuroraMiddleware/glibc
1
0
Fork 0
mirror of https://sourceware.org/git/glibc.git synced 2024-11-23 13:30:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

nss_db: fix endent wrt NULL mappings [BZ #24695] [BZ #24696]

Browse Source 
nss_db allows for getpwent et al to be called without a set*ent,
but it only works once.  After the last get*ent a set*ent is
required to restart, because the end*ent did not properly reset
the module.  Resetting it to NULL allows for a proper restart.

If the database doesn't exist, however, end*ent erroniously called
munmap which set errno.

The test case runs "makedb" inside the testroot, so needs selinux
DSOs installed.

(cherry picked from commit 99135114ba)
(note: tests excluded as test-in-container infrastructure missing)
This commit is contained in:
DJ Delorie 2019-06-28 18:30:00 -05:00
parent 0d3905b110
commit cef4c840a8
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ChangeLog
View File

@ -1,3 +1,11 @@
2019-07-10 DJ Delorie <dj@redhat.com>
Sergei Trofimovich <slyfox@inbox.ru>
[BZ #24696]
[BZ #24695]
* nss/nss_db/db-open.c (internal_endent): Protect against NULL
mappings.
2019-07-01 H.J. Lu <hongjiu.lu@intel.com>
[BZ #24259]

6
nss/nss_db/db-open.c
View File

@ -63,5 +63,9 @@ internal_setent (const char *file, struct nss_db_map *mapping)
void
internal_endent (struct nss_db_map *mapping)
{
munmap (mapping->header, mapping->len);
if (mapping->header != NULL)
{
munmap (mapping->header, mapping->len);
mapping->header = NULL;
}
}