diff options
author | vlakoff <vlakoff@gmail.com> | 2013-02-19 01:45:23 +0100 |
---|---|---|
committer | vlakoff <vlakoff@gmail.com> | 2013-02-19 01:45:23 +0100 |
commit | f399425bacfa461a77be2e841c3c1f48f9241c4a (patch) | |
tree | d66480130ff53820a475a9aed8827e6cdee63238 /system/libraries/Upload.php | |
parent | 6244086d6e251fc4b3c71d23da625ffb59b85d77 (diff) |
Fix a code comment in Upload->_file_mime_type()
Availability of dangerous functions is now tested using function_usable().
Diffstat (limited to 'system/libraries/Upload.php')
-rw-r--r-- | system/libraries/Upload.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/system/libraries/Upload.php b/system/libraries/Upload.php index acd76b03b..1c14f99ed 100644 --- a/system/libraries/Upload.php +++ b/system/libraries/Upload.php @@ -1212,7 +1212,7 @@ class CI_Upload { * Notes: * - the DIRECTORY_SEPARATOR comparison ensures that we're not on a Windows system * - many system admins would disable the exec(), shell_exec(), popen() and similar functions - * due to security concerns, hence the function_exists() checks + * due to security concerns, hence the function_usable() checks */ if (DIRECTORY_SEPARATOR !== '\\') { @@ -1223,7 +1223,7 @@ class CI_Upload { if (function_usable('exec')) { /* This might look confusing, as $mime is being populated with all of the output when set in the second parameter. - * However, we only neeed the last line, which is the actual return value of exec(), and as such - it overwrites + * However, we only need the last line, which is the actual return value of exec(), and as such - it overwrites * anything that could already be set for $mime previously. This effectively makes the second parameter a dummy * value, which is only put to allow us to get the return status code. */ |