in reply to Image uploading
For example, you seem to be chasing down all the possible bad stuff people could do with your script, and acting against them. This is known as "enumerating badness", and is generally acknowledged as an impossible task.
Instead, it is much better to write your code in such a way that all images are uploaded without execute permissions, for example, since they don't need them, and therefore it makes sense to "deny by default". And then you don't have to worry about people executing them.
Or, instead of finding out if a file is an image when uploading, "deny by default" execution permissions for any uploaded file, and add them later if a specific set of conditions is met (although I can't imagine letting random stuff uploaded to be executed is ever a good idea).
So it's better to allow *only* the stuff you want to allow, than to try to deny every single bad thing that could happen.
I do hope that helps.
|
---|