|
Bugzilla – Full Text Bug Listing |
| Summary: | VUL-0: CVE-2004-1488: multiple wget flaws | ||
|---|---|---|---|
| Product: | [Novell Products] SUSE Security Incidents | Reporter: | Ludwig Nussel <lnussel> |
| Component: | Incidents | Assignee: | Marcus Meissner <meissner> |
| Status: | RESOLVED FIXED | QA Contact: | Security Team bot <security-team> |
| Severity: | Normal | ||
| Priority: | P3 - Medium | CC: | mls, security-team |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | CVE-2004-1488: CVSS v2 Base Score: 5.0 (AV:N/AC:L/Au:N/C:N/I:P/A:N) | ||
| Found By: | Other | Services Priority: | |
| Business Priority: | Blocker: | --- | |
| Marketing QA Status: | --- | IT Deployment: | --- |
| Attachments: |
the patch for ".."
New patch handling .. issue wgettrap.pl |
||
|
Description
Ludwig Nussel
2004-12-13 21:05:45 UTC
*** Bug 64178 has been marked as a duplicate of this bug. *** This is verdammt nasty. There is no fix yet, I'll keep an eye on for a fix. any news? No. ... and still no patch. :( i'll ask the vendor-sec folks for a patch. Mauro Tortonesi <mtortonesi@ing.unife.it> is the new maintainer of wget. mmj, do you like to ask him? CAN-2004-1487 - overwrite files via ".." in path component
CAN-2004-1488 - missing quote for control characters can scrunch the display
is there any fix from the author yet? Did anyone contact him? If not I can try to make a fix. Sebastian, we dont have a fix yet. Please try to make one. Thanks. Created attachment 28641 [details]
the patch for ".."
This is my proposal. Wget already has a translation
table for characters like ../ but as I understod the bug
if the hostname contains evil characters it fails
to clean them. I substiutute these character-sequences by '_'.
For the control-character output I have no clue.
Ok, do we need to do an update, or is fixing in STABLE enough? Dont know. Thomas? We are still missing a patch for these control-character stuff. If we want to fix it at all. Might be hard to wrap all printfs() there? Yes the control-char bug is just the tip of the iceberg and constraints will change with new terminal types... so let's forget about it. But CAN-2004-1487 should be fixed for all affected versions since 8.2. Thanks for the patch, BTW. Can I get a SWAMP ID here please? gimmie a minute... SM-Tracker-445 Ok, submitted for all versions, including patchinfo. SLES9 is still pending due to Ralf deciding whether or not he wants to fix the bad wget in SLES9 and thus lose LFS support or not [ bug #47965 and bug #51031 ] QA-Team: Please test wget very well because it was used ba YOU in the past. :) I'll submit patchinfo files after the SLES9 case was finished. needs a decision from you, Ralf. ralf declined the rollback for now. please submit a wget without the rollback done/SLES9 you can submit the current vcersion to done/SLES9-SP2 for SP2 inclusion. autobuild / mls did not like the patch: - it does replace ".." unconditionally, even within filenames. => Use "/.." at least. - Suggestion from MLS is too: The redirect handling should not change the filename retrieved at all, this would be the prefered fix for this problem. => redesign patch :/ Further comments: I don't think the "don't change the filename" is possible, so it's ok with me if ".." isn't unconditional replaced, e.g. "foo..bar" still works. Hmm, another idea: set a flag if you do a redirect and only sanitize the path if a redirect occurred, so users could still get dot files. The .. replace was meant that way. Whats the matter with someone creating strange ".." files? Theres also a translation table for other characters. If you rather like /.. to be replaced, do it, its just one character fix to the fix :-) I remember the "fetch" utility from BSD has a similar behaivior, it dislikes strange dots too. Don't you think that your patch will break existing applications? Ok, I thought this was blessed by vendor-sec? When I sent it around, at least nobody complained. Created attachment 31702 [details]
New patch handling .. issue
This is exectly the same patch except it matches /..
except only ..
Please check in. Patchinfos can be the same. Plop plop. Please handle :) the wget maintainer was no longer in cc or assigned.. mmj there is a new patch attached. But I think we can even get rid of the "/.." cases since it is handled by the "/." case already. can you submit updated packages please taking care of it. Created attachment 33979 [details]
wgettrap.pl
perl script that was previously inlined now attached
we released an update, but fixing the directory traversal issue and dot file overwrites issues only. CVE-2004-1488: CVSS v2 Base Score: 5.0 (AV:N/AC:L/Au:N/C:N/I:P/A:N) |