Closed #1495.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1495#event-4351140494___
Rpm-maint mailing list
Oh, right, I'd forgotten about that. That existing check is wrong in it's
current habitat, it got moved around in the last big refactoring. A header is
fundamentally nothing but an abstract data container, and being empty is an
entirely valid condition for it to be in.
--
You are receiving
We actually already reject empty headers (with no tags at all) in
`hdrblobVerifyRegion`. So the only question remaining is if an empty region
(`ril == 0`) is valid.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
It's entirely legitimate for a *header* to be empty, region or no.
A *package* cannot have an empty header (or region), but that'd be a different
kind of check.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Such headers are useless and might cause problems elsewhere.
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/1495
-- Commit Summary --
* Forbid headers with only a region
-- File Changes --
M lib/header.c (6)
--
