Closed Bug 132722 Opened 22 years ago Closed 22 years ago

Bug in url_decode.

Categories

(Bugzilla :: Bugzilla-General, defect)

Product:
Bugzilla
Component:
Bugzilla-General
Version:
2.10
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: terry, Assigned: justdave)

Details

Attachments

(1 file)

Patch to fix bug in url_decode
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...

    
    

    
  
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-

    
    

    
  
Never mind.  I am a bucket-head.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → INVALID
QA Contact: matty_is_a_geek → default-qa

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: