diff options
| author | Stef Walter <stef@thewalter.net> | 2013-07-17 18:55:38 +0200 |
|---|---|---|
| committer | Stef Walter <stef@thewalter.net> | 2013-07-18 13:04:37 +0200 |
| commit | 1548d82560b242579f5ba216b66bd59ccd0f3fd0 (patch) | |
| tree | 2168b80fb86a9500514706f5402038976061e196 /common | |
| parent | 29a5df009656dc09be781c4939cec3613a0a12cb (diff) | |
Don't call memdup with zero length or NULL pointer
https://bugzilla.redhat.com/show_bug.cgi?id=985433
Diffstat (limited to 'common')
| -rw-r--r-- | common/attrs.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/common/attrs.c b/common/attrs.c index b06a342..9f3e522 100644 --- a/common/attrs.c +++ b/common/attrs.c @@ -143,8 +143,13 @@ attrs_build (CK_ATTRIBUTE *attrs, } memcpy (attr, add, sizeof (CK_ATTRIBUTE)); - if (!take_values) - attr->pValue = memdup (attr->pValue, attr->ulValueLen); + if (!take_values && attr->pValue != NULL) { + if (attr->ulValueLen == 0) + attr->pValue = malloc (1); + else + attr->pValue = memdup (attr->pValue, attr->ulValueLen); + return_val_if_fail (attr->pValue != NULL, NULL); + } } /* Mark this as the end */ |