rpmbuild allows names and versions/releases like: ../../../

[Jeff Johnson]

tracker

[seth (seth-redhat-bugs)]

Description of problem:
look at this spec:
http://skvidal.fedorapeople.org/misc/foo-1.1.spec

build that spec.
note where the files end up being written/built.

cry softly into your keyboard when you think about %clean

<shudder>
so I think maybe ~, /, and .. among anything else you can think of need to be stricken from pkg name/ver/rel fields?

[Jindrich (jindrich-redhat-bugs)]

Fixed upstream.

~,/ and .. are no more permitted in NVR.

[Panu (panu-redhat-bugs)]

Fixed in rawhide now, keeping open for F10 tracking.

[Tomas (tomas-redhat-bugs)]

Removing group restriction.

[Jeff (jeff-redhat-bugs)]

FYI: the issue of wild characters in Name: was reported to a vendor-sec representative
in December and fixed @rpm5.org by adding PCRE validation patterns for all tags, not
just spot checking NVR. The issue is considerably more complex than, say,
   Name: ~;
and can be exercised by any script, not just rpmbuild, that constructs file paths
from RPM package tags.

Not that vendor-sec is worth much these days ...

[Jeff (jeff-redhat-bugs)]

Um, these issues are hardly fixed by preventing '/' and '~' and '..'.

After 30 minutes of dinking, there are (some) of the flaws I found (note
that I'm not even a professional "black hat", I'm quite sure that
additional paranoia and malice might have reduced 30 minutes
to something much less).

For starters, ~ is unnecessary when
   Name: whatever;cd;
is possible.

And there are eval contexts with '<' available that make
failing a build if/when a '/' is present pointless:

    Name: whatever;cd;cat<$SHELL>$'057'tmp$'057'myshell

Note that similar but much more serious exploits are possible
using /dev/tcp/HOSTNAME/PORT importing (and executing) a rootkit.

I suggest that you commit to permitted character sets in specific tag contents
and undertake explicit verification rather than fooling yourself with silliness like

    But you can't mention '/' or '~' or ".." in name/version/release!

instead. It really isn't __THAT__ hard to add an explicit permitted character set
in tag content like RPMTAG_NAME and RPMTAG_VERSION etc etc.

YMMV. Have fun!

[Fedora (fedora-redhat-bugs)]

rpm-4.6.1-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/rpm-4.6.1-1.fc10

[Fedora (fedora-redhat-bugs)]

rpm-4.6.1-1.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with
 su -c 'yum --enablerepo=updates-testing update rpm'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-5214

[Fedora (fedora-redhat-bugs)]

rpm-4.6.1-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.