[Rpm-maint] [rpm-software-management/rpm] rpm is unable to find existing group (>= 4.19.x regression) (Issue #3594)

[Jan Palus]

jpalus left a comment (rpm-software-management/rpm#3594)

One more observation regarding `lookup_field()`:
https://github.com/rpm-software-management/rpm/blob/b043604b9eb684dc761aeacf55a784632ca0ebcd/lib/rpmug.cc#L56-L92

When matching record is found we don't `break` the `while` loop iterating over all lines:
https://github.com/rpm-software-management/rpm/blob/b043604b9eb684dc761aeacf55a784632ca0ebcd/lib/rpmug.cc#L81-L86

Whether that's correct or not depends on whether `lookup_field()` is supposed to support multiple entries with the same key and return "latest" entry. If it shouldn't `break` is missing. If it should there's a memory leak since previously allocated memory assigned to `*rc` is not freed.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3594#issuecomment-2685530783
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/issues/3594/2685530783 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250226/66575e43/attachment-0001.htm>