Avoid crash copying empty hash table.

Visible with typeset -p.
This commit is contained in:
Stephane Chazelas 2017-12-22 22:17:09 +00:00 committed by Peter Stephenson
parent 1e46f73b8e
commit c2cc8b0fbe
2 changed files with 9 additions and 4 deletions

View file

@ -1,5 +1,7 @@
2018-01-04 Peter Stephenson <p.stephenson@samsung.com>
* Stephane: 42159: Src/params.c: avoid crash copying empty hash table.
* Sebastian: 42188: Src/Modules/system.c: It is necessary to
close the lock descriptor in some failure cases.

View file

@ -549,10 +549,13 @@ scancopyparams(HashNode hn, UNUSED(int flags))
HashTable
copyparamtable(HashTable ht, char *name)
{
HashTable nht = newparamtable(ht->hsize, name);
outtable = nht;
scanhashtable(ht, 0, 0, 0, scancopyparams, 0);
outtable = NULL;
HashTable nht = 0;
if (ht) {
nht = newparamtable(ht->hsize, name);
outtable = nht;
scanhashtable(ht, 0, 0, 0, scancopyparams, 0);
outtable = NULL;
}
return nht;
}