Comment 6 for bug 1077603

Revision history for this message
Jeff Johnson (n3npq) wrote :

This is not an uninitialized string, a NULL pointer for a non-existent tag
iwhich isn't being checked for in poldek.

But that is a developer's guess: if you *REALLY* think uninitialized
string rather than a NULL pointer, then either examine variables
with gdb (or easier/better: run under valgrind).

Yes the Group: tag is optional @rpm5.org: previously Group: was mandatory, and
inheirited into subpkgs (iirc: this is all from memory).

poldek needs to be fixed to check return codes and also validate pointers != NULL.

The quickest/dirtiest fix is to use strdup("Unknown") to assign a Group: tag
value that wasn't in a header (or in specspo lookaside or other retrievals).

You can also patch rpm to make Group: tag mandatory again again again.