PHP: Security
up
down
2
steffen at morkland dot com
6 years ago
In Reply to djjokla and others
Consider placing all incude files as mentioned before in a seperate folder containing a .htaccess containing a Order Deny,Allow
the create a index file, which is intended to handle ALL request made to you php application, then call it with index.php?view=index
the index file could look a bit like this:
<?php
]){
case 'index':
include('libs/index.php');
break;
default:
include('libs/404.php');
break;
}
?>
this could be an array or something even more creative. it actually does'nt matter how you do it... running all pages through one central script has one big advantage.... CONTROL.
at any givin time, you can easily implement access control to functions without forgetting crucial files.
up
down
1
moehbass at gmail dot com
5 years ago
First, q much simpler solution to preventing people from viewing code inside of an includable file would be to give include file an extension that ends with php (e.g. myFile.inc.php).
Secondly, and more importantly, why on earth would you want to put program-level code in an include file? By that I mean something life this:
myFile.inc.php
--------------------------------
...
if ($var = 'whatever')
// connect to the database
else
// do something else.
--------------------------------
An include file should not contain logic! Rather, it is an encapsulated unit of code that should not do anything on its own unless asked to. To implement this ideology, consider including function definitions only in your include files, then once you include them in the script, call such functions from within your program (i.e. the script that included the inc file). If you don't know the names of the functions ab initio, use call_user_func() or call_user_func_array() and pass it the name of the function that's dependent on context.
If you MUST put program-level logic in your include files, consider simply putting it in the program!
Why should you consider this? How about variable name clashes for a starter! You can think of more, I am shure!
Hope that helped
up
down
0
nick dot hristov at gmail dot com
8 years ago
A correction to previous post by Dave Mink.
<Files ~ "\.inc$">
Order allow,deny
Deny from all
Satisfy All
</Files>
Will not stop something like
http://http://www.zjjv.com///includefile.inc?pointlessvar=blahblah
Here is something more sophisticated for this task:
<Location ~ "/[^ ](?=\.inc(\?[^ ]*)?)/">
Options None
Order Allow, Deny
Deny from All
AllowOverride None
Satisfy All
</Location>
Also, consider placing in your httpd.conf
<Location ~ "/[^ ](?=\.phps(\?[^ ]*)?)/">
Options None
Order Allow, Deny
Deny from All
AllowOverride None
Satisfy All
</Location>
up
down
0
ocrow at simplexity dot net
9 years ago
If your PHP pages include() or require() files that live within the web server document root, for example library files in the same directory as the PHP pages, you must account for the possibility that attackers may call those library files directly.
Any program level code in the library files (ie code not part of function definitions) will be directly executable by the caller outside of the scope of the intended calling sequence. An attacker may be able to leverage this ability to cause unintended effects.
The most robust way to guard against this possibility is to prevent your webserver from calling the library scripts directly, either by moving them out of the document root, or by putting them in a folder configured to refuse web server access. With Apache for example, create a .htaccess file in the library script folder with these directives:
Order Allow,Deny
Deny from any
up
down
0
9 years ago
For real security you should consider providing chrooted jail's for your users.
up
down
0
dangan at blackjaguargaming dot net
6 years ago
I'd recommend a 404 over a 403 considering a 403 proves there is something worth hacking into.
index.php:
<?php
define);
include('includes/include.sqlfunctions.php');
// Rest of code for index.php
?>
include.sqlfunctions.php (or other include file):
<?php
// Not identical to 1
{
);
echo "<!DOCTYPE HTML PUBLIC \"-//IETF//DTD HTML 2.0//EN\">\n<html><head>\n<title>404 Not Found</title>\n</head>";
echo ;
echo ;
// Echo output similar to Apache's default 404 (if thats what you're using)
exit;
}
// Rest of code for this include
?>
up
down
0
Thomas "Balu" Walter
7 years ago
Since many users can not modify apache configurations or use htaccess files, the best way to avoid unwanted access to include files would be a line at the beginning of the include-file:
?>
And in all files that are allowed to be called externally:
?>
Balu
up
down
-1
k
6 years ago
How about not putting the php code in the web-root at all...?
You can create a public directory with the css, html, etc and index.php there. Then use the include_path setting to point to the actual php code, eg...
webstuff
phpcode
public
images
css
index.php
then set the include path to "../phpcode" and, as php is executed from the directory of the script, all should be well.
I'd also call the main index "main.page", or something else, instead of "index.php" and change the web server default index page. That way you cant get hit by things trawlling the web for index pages.
up
down
-1
djjokla AT gmail dot com
6 years ago
If a single file has to be included than I use the following
index.php ( where the file is gonna be included )
___________
<?php
define);
include('folder/footer.inc.php');
?>
and the footer file (for example) looks this way then
footer.inc.php ( the file to be inluded )
___________
<?php
defined);
echo('Copyright to me in the year 2000');
?>
So when someone tries to access the footer.php file directly he/she/it will get the "Not with me my friend" messages written on the screen. An alternative option is to redirect the person who wants to access the file directly to a different location, so instead of the above code you would have to write the following in the footer.inc.php file.
<?php
defined);
echo('Copyright to me in the year 2000');
?>
In normal case a redirection to an external site would be annoying to the visitor, but since this visitor is more interested in hacking the site than in reading the content, I think it's only fair to create such an redirection. We dont' realy want someome like this on our sites.
For the file protection I use .htaccess in which I say to protect the file itself and every .inc file
<Files ~ "^.*\.([Hh][Tt]|[Ii][Nn][Cc])">
Order allow,deny
Deny from all
Satisfy All
</Files>
The .htaccess file should result an Error 403 if someone tries to access the files directly. If for some reason this shouldn't work, then the "Not with me my friend" text apears or a redirection (depending what is used)
In my eyes this looks o.k. and safe.
up
down
-1
7 years ago
that's cool, but i use this code right here:
<?if(!defined('IN_SCRIPT')){header('HTTP/1.0 404 not found');exit;}?>
adding a 404 header will not give the user any clue that the include-file even exists !!!
i also protect the whole include-directory with a .htaccess file that says: "Deny from all"
so i guess that's pretty secure
up
down
-1
annonymous at domain dot com
9 years ago
best bet is to build php as cgi, run under suexec, with chroot jailed users. Not the best, but fairly unobtrusive, provides several levels of checkpoints, and has only the detriment of being, well, kinda slow. 8)
up
down
-1
ManifoldNick at columbus dot rr dot com
9 years ago
Remember that security risks often don't involve months of prep work or backdoors or whatever else you saw on Swordfish ;) In fact one of the bigges newbie mistakes is not removing "<" from user input (especially when using message boards) so in theory a user could secerely mess up a page or even have your server run php scripts which would allow them to wreak havoc on your site.
up
down
-2
Lionel
7 months ago
Features for manipulation of input data:
- Additional or lack of form variables
It will send only pass variables from a form provided by the attacker set and not all the form variables like the real thing.
BSP:
<form ...>
<input>
<input>
<input>
$ _POST ['Name'] = "root";
$ _POST ['Password'] = "12345";
$ _POST ['Optional'] would be missing
- Session ID does not match the standard
Session ID is guessed or tried themselves together, making it too short or contains incorrect characters
BSP:
http://http://www.zjjv.com///?sid=ABCDabcd1234
instead of
http://http://www.zjjv.com///?sid=ABCD% 2Cabcd-1234
- Hidden, select, checkbox variables do not correspond to the model
The transmitted values are different from the predefined values of the above fields
BSP:
<select>
<option value="1"> One </ option>
<option value="2"> Two </ option>
<option value="3"> Three </ option>
</ select>
$ _POST ['Selection'] = 'a';
No integer value
- Source of variables (get, post, cookie) do not match
The attacker tries to parameter passing some variables to set or influence
BSP:
http://http://www.zjjv.com///?login=true
$ _GET ['Login'] = true
instead of
$ _COOKIE ['Login'] = true
