diff options
| author | dtucker <dtucker> | 2007-02-19 11:56:55 +0000 |
|---|---|---|
| committer | dtucker <dtucker> | 2007-02-19 11:56:55 +0000 |
| commit | 540cd0c912310af21d01e7dbf05940a34e6b2e51 (patch) | |
| tree | 4b850488abb32b5beec43d70da50596efd7ac28c /openbsd-compat/getrrsetbyname.c | |
| parent | 5e11a3e39c82e2d327c4753c971cc3c303fc50d9 (diff) | |
| download | openssh-540cd0c912310af21d01e7dbf05940a34e6b2e51.tar.gz | |
- (dtucker) [openbsd-compat/getrrsetbyname.c] Don't attempt to calloc
an array for signatures when there are none since "calloc(0, n) returns
NULL on some platforms (eg Tru64), which is explicitly permitted by
POSIX. Diagnosis and patch by svallet genoscope.cns.fr.
Diffstat (limited to 'openbsd-compat/getrrsetbyname.c')
| -rw-r--r-- | openbsd-compat/getrrsetbyname.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/openbsd-compat/getrrsetbyname.c b/openbsd-compat/getrrsetbyname.c index 6c86e02c..07231d00 100644 --- a/openbsd-compat/getrrsetbyname.c +++ b/openbsd-compat/getrrsetbyname.c @@ -303,10 +303,12 @@ getrrsetbyname(const char *hostname, unsigned int rdclass, } /* allocate memory for signatures */ - rrset->rri_sigs = calloc(rrset->rri_nsigs, sizeof(struct rdatainfo)); - if (rrset->rri_sigs == NULL) { - result = ERRSET_NOMEMORY; - goto fail; + if (rrset->rri_nsigs > 0) { + rrset->rri_sigs = calloc(rrset->rri_nsigs, sizeof(struct rdatainfo)); + if (rrset->rri_sigs == NULL) { + result = ERRSET_NOMEMORY; + goto fail; + } } /* copy answers & signatures */ |