Don't test for NULL when it is both unnecessary (the pointer is checked
against NULL when it is first allocated) and pointless (we've already dereferenced the pointer several times). Found by: Coverity Prevent(tm) CID: 3204
This commit is contained in:
parent
51bbfbd15c
commit
3662c7b8ad
@ -1168,7 +1168,7 @@ pax_header(struct archive_read *a, struct tar *tar, struct archive_entry *entry,
|
||||
return (-1);
|
||||
while (*wp && *wp != L'=')
|
||||
++wp;
|
||||
if (*wp == L'\0' || wp == NULL) {
|
||||
if (*wp == L'\0') {
|
||||
archive_set_error(&a->archive, ARCHIVE_ERRNO_MISC,
|
||||
"Invalid pax extended attributes");
|
||||
return (ARCHIVE_WARN);
|
||||
|
Loading…
x
Reference in New Issue
Block a user