Bugzilla – Attachment 40245 Details for
Bug 12552
Odd error caused by URI Escape
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12552 - Odd error caused by URI Escape
Bug-12552---Odd-error-caused-by-URI-Escape.patch (text/plain), 1.30 KB, created by
Kyle M Hall
on 2015-06-17 15:30:18 UTC
(
hide
)
Description:
Bug 12552 - Odd error caused by URI Escape
Filename:
MIME Type:
Creator:
Kyle M Hall
Created:
2015-06-17 15:30:18 UTC
Size:
1.30 KB
patch
obsolete
>From 7e6b8c243a9ee3f45c8c81cb3e9f88529a00fa6a Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Tue, 16 Jun 2015 21:07:58 -0400 >Subject: [PATCH] Bug 12552 - Odd error caused by URI Escape > >uri_escape: Unmatched [ in regex; marked by <-- HERE in m/([ <-- HERE ])/ at (eval 133) line 1. > >The issue is the use of uri_escape at line 174 of members/pay.pl > >Test Plan: >1) Apply this patch >2) Attempt to pay a fine with the "Pay" button >3) No regressions should be noted >--- > members/pay.pl | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > >diff --git a/members/pay.pl b/members/pay.pl >index 1ac140d..5512dc4 100755 >--- a/members/pay.pl >+++ b/members/pay.pl >@@ -177,7 +177,7 @@ sub redirect_to_paycollect { > $redirect .= get_for_redirect( 'notify_id', "notify_id$line_no", 0 ); > $redirect .= get_for_redirect( 'notify_level', "notify_level$line_no", 0 ); > $redirect .= get_for_redirect( 'accountlines_id', "accountlines_id$line_no", 0 ); >- $redirect .= q{&} . 'payment_note' . q{=} . uri_escape_utf8( $input->param("payment_note_$line_no") ); >+ $redirect .= q{&} . 'payment_note' . q{=} . uri_escape_utf8( quotemeta( $input->param("payment_note_$line_no") ) ); > $redirect .= '&remote_user='; > $redirect .= $user; > return print $input->redirect($redirect); >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 12552
: 40245