Closed
Bug 132722
Opened 23 years ago
Closed 23 years ago
Bug in url_decode.
Categories
(Bugzilla :: Bugzilla-General, defect)
Tracking
()
RESOLVED
INVALID
People
(Reporter: terry, Assigned: justdave)
Details
Attachments
(1 file)
|
896 bytes,
patch
|
justdave
:
review-
|
Details | Diff | Splinter Review |
I was just stealing the code for url_decode for some other project, when I
happened to notice a little mistake in the code. If a URL contains more
than one "+", it will fail to decode all of them into spaces.
I will attach the (extremely trivial) patch.
This bug has been there ever since Bugzilla was originally ported to perl...
|
Reporter | |
Comment 1•23 years ago
|
||
|
Assignee | |
Comment 2•23 years ago
|
||
Comment on attachment 75530 [details] [diff] [review]
Patch to fix bug in url_decode
tr/// doesn't take a /g parameter. This won't compile. tr/// should get all
occurances by default. If it's not, something else is broken.
Attachment #75530 -
Flags: review-
|
|
Reporter | |
Comment 3•23 years ago
|
||
Never mind. I am a bucket-head.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → INVALID
|
||
Updated•13 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•