I heard a rumor that when inserting binary data (files and such) into MySQL, you should use the bin2hex()
function and send it as a HEX-coded value, rather than just use mysql_real_escape_string
on the binary string and use that.
// That you should do
$hex = bin2hex($raw_bin);
$sql = "INSERT INTO `table`(`file`) VALUES (X'{$hex}')";
// Rather than
$bin = mysql_real_escape_string($raw_bin);
$sql = "INSERT INTO `table`(`file`) VALUES ('{$bin}')";
It is supposedly for performance reasons. Something to do with how MySQL handles large strings vs. how it handles HEX-coded values
However, I am having a hard time confirming this. All my tests indicate the exact oposite; that the bin2hex
method is ~85% slower and uses ~24% more memory.
(I am testing this on PHP 5.3, MySQL 5.1, Win7 x64 - Using a farily simple insert loop.)
For instance, this graph shows the private memory usage of the mysqld process while the test code was running:
Private Bytes used by the mysqld process http://atli.advefir.com/images/priv_mem_cropped.gif
Does anybody have any explainations or reasources that would clarify this?
Thanks.
I've been testing this myself, and I've come up with pretty consistent results. (Even though my tests are a tad crude.)
I've tested three computers
So far the tests on all three platforms have indicated the same tings:
bin2hex
into aX'...'
) uses more memory, on average, than using an escaped binary string (mysql_real_escape_string
on the raw data). - This seems true for both MyISAM and InnoDB.The test was basically just a simple loop that escaped or hex-coded the raw data (a 2.4 MiB image retrieved once at the top of the script), constructed the query string, and executed it via the
mysql_query
ormysqli::query
functions. - I tested with both extensions. Didn't seem to be any difference.I put the results from the Ubuntu 10.04 (#3) up in spreadsheets. The results from the Ubuntu 9.10 (#2) machine were pretty much the same, so I didn't bother set them up:
(Finally an excuse to test the Google Docs thing properly! xD)
These graphs show the private memory usage by the
mysqld
process on the Win7 (#1) machine.for example if you encounter similar problem as described here: http://www.php.net/manual/en/function.mysql-real-escape-string.php#82015
e.g. even though mysql_real_escape_string seems to be "binary-safe" you can't use it (as an example) in combination with igbinary_serialize - unserializing will just fail.
in that case you need to bin2hex prior inserting the data into mysql.
Also, usually you read more often data from mysql than inserting :)
A hex string is significantly longer than the corresponding binary string. Simply the transfer time and the copying it around inside PHP's and MySQL's memory may do the trick.
In all honesty I'm not expert on the underlying implementation, but wouldn't it be better to not pass the data inside the SQL at all, but using e.g.
PDOStatement
's parameter binding? Maybe someone more knowledgeable here can confirm whether that will indeed cause the data to be sent as a binary string, outside any SQL statement at all, or whether PDO just does the escaping and query string manipulation under the hood.Either way, you get a security (and simplicity) benefit right there.
This sounds like an urban legend to me.
bin2hex()
maps each byte in the input to two bytes in the output ('a'
->'61'
), so you should notice a significant memory increase of the script performing the query - it should use at least as much memory more as the byte length of the binary data to be inserted.Furthermore, this implies that running
bin2hex()
on a long string takes much longer than runningmysql_real_escape string()
, which - as explained in MySQL's documentation - just escapes 6 characters:NULL
,\r
,\n
,\
,,
and 'Control-Z'.That was for the PHP part, now for MySQL: The server needs to do the reverse operation to store the data correctly. Reversing either of the functions takes almost as long as the original operation - the reverse function of
mysql_real_escape_string()
needs to replace escaped values (\\
) with unescaped ones (\
), whereas the reverse ofbin2hex()
would need to replace each and every byte tuple with a new byte.Since calling
mysql_real_escape_string()
on binary data is safe (according to MySQL's and PHP's documentation or even when just considering that the operation does not do any other conversions than the ones listed above), it would make absolutely no sense to perform such a costly operation.