[RESOLVED] PHP isn't outputting my images correctly

TheoGB

Hi,

I have a site in Ruby on Rails where the images are stored in the MySQL database and Ruby outputs them all fine. Here's one:

http://londonprogressivejournal.com/image/picture/2

The problem is that I want to use the same MySQL blob data in PHP. Now if I retrieve it and use code as so:

<?php

if(isset($_GET['id']) && isset($_GET['type'])) 
{
	// Includes my DB connection info and my definitions for run_query
	include('_include_all.php');

$id    = $_GET['id'];
$type  = $_GET['type']; // table name
$query = "SELECT * FROM `".$type."` WHERE id = %d";

$result = run_query($query, $id);

header("Content-type: ".$result[0]['content_type']);
echo $result[0]['picture'];
}

?>

I get this: http://theogb.com/archive/Iraq.jpg

I've seen this sort of garbage output from the GD on PHP before.

I'm using XAMPP 1.7.0 but I've not had this sort of problem with other image outputs yet.

bradgrafelman

If you're just outputting binary image data from a MySQL table, then GD doesn't have anything to do with the problem.

Can you give us a link to the PHP page that's outputting the image?

TheoGB

Um, not easily. As I say, it's on my XAMPP. I'll try to get an uploaded version working.

TheoGB

Okay, not sure what it'll show you but:

http://londonprogressivejournal.com/db_file_out.php?id=2&type=images

TheoGB

Ah, I've solved it:

I was doing a 'stripslashes' to the blob data and this was corrupting it. Removing it made everything come out fine.

Interestingly everything I've read implies you should be doing an addslashes to the data when you retrieve it.

Any pointers on this area would be helpful as I've never used a BLOB data type before.

bradgrafelman

TheoGB wrote:
Interestingly everything I've read implies you should be doing an addslashes to the data when you retrieve it.

Not sure where you've read that - you should never have to use stripslashes() or addslashes() when adding/retrieving data from a database. The only DB-related modification you should be making is to use [man]mysql_real_escape_string/man (or the equivalent for whichever DB extension you're using) when preparing to store the data.

TheoGB

Yeah, I'm sure I was using that actually. I must have picked up an old version of my database routine as I don't even remember having stripslashes being called any more.

I have a link to the tutorial I was looking at somewhere. I didn't use addslashes since it seemed silly to me hence my checking about this.

Right, glad this is solved.

Cheers.