Skip to content

CGI 4 send alerts for the use of param.#6

Open
denics wants to merge 2 commits intoeprintsug:mainfrom
denics:master
Open

CGI 4 send alerts for the use of param.#6
denics wants to merge 2 commits intoeprintsug:mainfrom
denics:master

Conversation

@denics
Copy link
Contributor

@denics denics commented Jul 25, 2015

This fix #5

denics added 2 commits July 25, 2015 19:57
Calling param() in list context can lead to vulnerabilities if you do not sanitise user input as it is possible to inject other param keys and values into your code. This is why the multi_param() method exists, to make it clear that a list is being returned, note that param() can still be called in list context and will return a list for back compatibility. Source: http://search.cpan.org/~leejo/CGI-4.21/lib/CGI.pod#Fetching_the_value_or_values_of_a_single_named_parameter
@ghost ghost force-pushed the master branch 2 times, most recently from ea41995 to 278470c Compare May 22, 2018 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Security error in fresh installation

1 participant