Just ignore miFreeHeader() return on rpmdbNextIterator() too
- There's no meaningful way to return a write-error here, rpmdbNextIterator() caller wouldn't know whether NULL at termination is early due to error or not. So just ignore the return from miFreeHeader(), it can only fail on rewrite mode (ie markReplacedFiles()) so impact is rather limited anyway.
This commit is contained in:
parent
29fffe9aa5
commit
96c4f818a5
|
@ -1679,7 +1679,6 @@ Header rpmdbNextIterator(rpmdbMatchIterator mi)
|
|||
void * keyp;
|
||||
size_t keylen;
|
||||
int rc;
|
||||
int xx;
|
||||
|
||||
if (mi == NULL)
|
||||
return NULL;
|
||||
|
@ -1779,7 +1778,7 @@ top:
|
|||
}
|
||||
|
||||
/* Rewrite current header (if necessary) and unlink. */
|
||||
xx = miFreeHeader(mi, dbi);
|
||||
miFreeHeader(mi, dbi);
|
||||
|
||||
/* Is this the end of the iteration? */
|
||||
if (uh == NULL)
|
||||
|
|
Loading…
Reference in New Issue