PHP Classes

Are you serious?

Recommend this page to a friend!

      PHP Classes blog  >  PHP security exploit ...  >  All threads  >  Are you serious?  >  (Un) Subscribe thread alerts  
Subject:Are you serious?
Summary:it's NOT a PHP security exploit...
Messages:9
Author:Gonzalo
Date:2007-06-21 03:44:04
Update:2009-10-08 19:11:34
 

  1. Are you serious?   Reply   Report abuse  
Picture of Gonzalo Gonzalo - 2007-06-21 03:58:26
it's an "stupid developer" exploit...
What are you expecting from PHP developers? "Fatal Error: upload of php files is not available for security reasons"

Last release: Microsoft PHP 5.2.1 ¬¬

  2. Re: Are you serious?   Reply   Report abuse  
Picture of Manuel Lemos Manuel Lemos - 2007-06-21 04:03:05 - In reply to message 1 from Gonzalo
I would not use such harsh adjectives on PHP developers, but the lack of education on security aspects makes some developers open holes on their sites inadvertently.

The point of this article is to open the eyes of developers that have not thought about these security aspects before.

  3. Re: Are you serious?   Reply   Report abuse  
Picture of Manuel Lemos Manuel Lemos - 2007-06-21 04:16:04 - In reply to message 1 from Gonzalo
BTW, the problem is not allowing or not allowing PHP files to be uploaded.

The problem is that many PHP developers use the getimagesize function to validate uploaded image sizes. A valid GIF file can embed PHP code and would not be detected as invalid by the getimagesize function.

As mentioned in the article, this would not be a problem if the developers would not make the image files available for access directly with the original file names. If the file name is .php or .php3 or anything that Apache processes as a PHP request, sites may be abused.

Even when the file name extension is validated, some developers use the include or require functions to serve the images. In that case, the sites may also be abused.

As you may see, the developers do not need to be so stupid to make such security mistakes.

  4. Re: Are you serious?   Reply   Report abuse  
Picture of fate fate - 2007-06-21 21:56:01 - In reply to message 3 from Manuel Lemos
erm Im not claiming to be a great coder or such, But I did have an issue like this of people attempting to upload double extention filenames, I run a simply script to parse the file name and check last extention for validation This may seem simple or such but it works...

  5. Re: Are you serious?   Reply   Report abuse  
Picture of Manuel Lemos Manuel Lemos - 2007-06-21 22:03:28 - In reply to message 4 from fate
That may not be sufficient depending on how you serve the files to the users.

As mentioned in the article, if you use include or require to serve the uploaded images, you are open to security exploits. It may sound silly but some developers do that.

The problem is that valid GIF files may still contain by sequences inside of them like this <?php readfile('/etc/passwd'); ?> . Once PHP encounters these bytes in the GIF file that is included, that code is run.

Using getimagesize() to validate the image file would not prevent the problem because the file is a valid GIF image.

There are reports of sites being abused with exploits like this. So, depending on cautious you are, you may or not be subject to this exploit.

  6. Re: Are you serious?   Reply   Report abuse  
Picture of genshenghu genshenghu - 2007-08-31 08:06:43 - In reply to message 5 from Manuel Lemos
study!

  7. Re: Are you serious?   Reply   Report abuse  
Picture of David David - 2009-07-20 18:36:23 - In reply to message 5 from Manuel Lemos
Sorry for digging such an old topic, but why on earth would someone use include() or require() to serve a picture? Sounds a bit like "hack me please, btw here is my password" security practice...

  8. Re: Are you serious?   Reply   Report abuse  
Picture of Manuel Lemos Manuel Lemos - 2009-07-20 19:40:30 - In reply to message 7 from David
As it was mentioned above, it may sound silly, but some people do that because they do not know a better way to include a file in the current script output.

  9. Re: Are you serious?   Reply   Report abuse  
Picture of starbuck starbuck - 2009-10-08 19:11:34 - In reply to message 7 from David
"Sorry for digging such an old topic, but why on earth would someone use include() or require() to serve a picture? Sounds a bit like "hack me please, btw here is my password" security practice..."

Actually, while the threat is correctly identified (someone uploading GIF pictures with injected php code), the vector isn't.

The scenario where developers use include/require on gif pictures mostly aims at dealing with retards. Such measures won't help them protecting their website. This is due to the very simple fact that someone who executes gif pictures through require/include API calls needs a lot more training than just reading posts about PHP security.

The most probable scenario, however, occurs when websites are hosted on large public hosting servers, managed by weak admins dealing with retard users and they get forced to configure all extensions to pass through the PHP parser (this happens,believe me)

Another situation is when you need to enforce proper access control on people who directly type the URL of confidential documents, such as .docx, .pdf or .gif documents. Some developers chose a "parse all" strategy to make sure there is always some script that gets executed.


These are the 2 additional but nontheless, important, vectors that lead to GIF images upload being an exploit tool.

The most appropriate measure consists of properly configuring the web server, ie: disabling script execution within the upload folders. With this measure implemented, even weak validation code cannot turn an uploaded document into a server-side executed bomb. However, proper code still helps and the author's recommendations should be followed carefully.

Hope it helps,
sb