Fatal Error: Allowed memory size of...

slightlyeskew

Hey all!

So I received this error:

Fatal error: Allowed memory size of 16777216 bytes exhausted (tried to allocate 14942223 bytes) on line 98

How do I adjust to allow for more allocated memory? (I don't think I should be reaching this limit in the first place but if I can easily correct this who cares! 🙂 )

require 'Mail.php';
require 'Mail/mime.php';
require_once 'MDB2.php';
$db = MDB2::connect('mysql:');
if (MDB2::isError($db)) { die("Can't connect: " . $db->getMessage()); }

$mime = new Mail_mime;$text = 'Your Email Client is not configured to receive HTML.';$mime->setHTMLbody($text);
$id = 1;

$firstLastNames = $db->query("SELECT First_Name,Last_Name,Email_Address FROM ShareholderNames WHERE ShareholderID = $id");
$nameArray = $firstLastNames->fetchAll();

$first = $nameArray[0][0];$last = $nameArray[0][1];$email = $nameArray[0][2];


//$to = $email;$headers['Subject'] = 'Stock Report';
$to = ' @gmail.com'; $headers['From']='';$headers['Subject'] = 'Test';

$currentPrice = $db->queryOne("SELECT Share_Price FROM TransactionHistory ORDER BY Transaction_Date DESC LIMIT 1");

$uberPull = $db->query("SELECT TransactionID, Source_Shareholder, Destination_Shareholder, Source_Certificate, Destination_Certificate,Transaction_Date,Share_Price, Number_Of_Shares,Transaction_Notes FROM TransactionHistory WHERE Destination_Shareholder = $id OR Source_Shareholder = $id ORDER BY TransactionID");

$uberArray = $uberPull->fetchAll();$numberofUber = count($uberArray);
$nameArray[0]['Total Shares'] = 0;$totalShares = $nameArray[0]['Total Shares'];	

$uberCounter2 = 0;//this is for detailed.
$detailedCounter = 0;

$html = '<html><body><b>'.$last.', '.$first.' '.$id.'</b><br/><br/>';//this works
$html .= '<table border="border" cols="9"><th>Selling Certificate</th><th>Buying Certificate</th><th>Seller</th><Buyer<th></th><Price Per Share</th><th>Number of Shares</th><th>Transaction Price</th><th>Transaction Date</th><th>Transaction Notes</th></tr>';

while($uberCounter2 < $numberofUber)
{
$sourceCert = $uberArray[$uberCounter2][3];$destCert = $uberArray[$uberCounter2][4];$numOfShares = $uberArray[$uberCounter2][7];$transPricePerShare = $uberArray[$uberCounter2][6]; $transPrice = $numOfShares * $transPricePerShare; $buyer = $uberArray[$uberCounter2][4];$seller = $uberArray[$uberCounter2][3];$transNotes = $uberArray[$uberCounter2][8];$transDate = $uberArray[$uberCounter2][5];

$transPrice = number_format($transPrice,2);$transPricePerShare = number_format($transPricePerShare,2);$numOfShares = number_format($numOfShares);


if($seller == 0)
{$seller = 'Treasury';}
if($buyer == 0)
{$buyer = 'Treasury';}
if($sourceCert == 0)//
{$sourceCert = 'Treasury Certificate';}
if($destCert == 0)
{$destCert == 'Treasury Certificate';}
$html .= '<tr>';
$html .= '<td>'.$sourceCert.'</td><td>'.$destCert.'</td><td>'.$seller.'</td><td>'.$buyer.'</td><td>$'.$transPricePerShare.'</td>';
$html .= '<td>'.$numOfShares.'</td><td>$'.$transPrice.'</td><td>'.$transDate.'</td><td>'.$transNotes.'</td></tr>';
$uberCounter++;
}
$html .= '</table><br/><br/>';

$mime->setHTMLBody($html);$body= $mime->get();$headers = $mime->headers($headers);


$host = '';$username = '';$password = '';
$smtp =& Mail::factory('smtp', array ('host' => $host, 'auth' => true, 'username' => $username,'password' => $password));
$smtp->send($to,$headers,$body);

echo $html;

if (PEAR::isError($smtp)) {
	echo("<p>" . $mail->getMessage() . "</p>");
	} else {
	echo("<p>Message successfully sent!</p>");
	}

Thanks for your help!

-slight

slightlyeskew

Additionally, I'm running Ubuntu powered via Virtual Box. Currently I've got aprox 1500 MB allocated to VB. Thanks!

slightlyeskew

Additionally, as a frequent newbie poster I know someone is going to tell me that I should have error_reporting on 🙂.

With it turned on I don't receive any additional errors other than mention of one undefined variable.

Thanks!

bradgrafelman

Ouch.. a memory limit of only 16MB?? The default for a new installation of PHP is something like 128 MB, IIRC. I'm assuming you have access to the php.ini file, correct? If so, open it and search for the "memory_limit" directive in there - change it to something like 64M or 128M.

EDIT: Also note that you could try to write code that uses memory more efficiently. For example, after this code:

$nameArray = $firstLastNames->fetchAll(); 

$first = $nameArray[0][0];$last = $nameArray[0][1];$email = $nameArray[0][2];

You're left with an array and a SQL resource that you never use again. As for the former of the two, you could either a) use an associative array e.g. $nameArray[0]['First_Name'] rather than duplicate the data in separate variables, or b) use [man]list/man to create the variables without an intermediate array, e.g.:

list($first, $last, $email) = $firstLastNames->fetchRow();

As for the second, you could call the "free()" method and re-use the same variable to execute queries rather than creating new ones for each query, though I don't think that resource pointers take up too much memory.

slightlyeskew

Thanks Brad!

I found the line in php.ini. It was indeed set to 16M. However I changed the line to 128M and it is still bringing up the same error.

Now it says: "Allowed memory size of 134217728 bytes exhausted (tried to allocate 132382693 bytes)" I could keep upping this...but is this kind of behavior unusual? Is there something ridiculous about my script? As an fyi, the variable "numberofUber" could be somewhere around 1000. What's the max memory that I can safely allocate to php.ini?

Thanks so much for your time!

bradgrafelman

Er... it still blew up with 128M as the max memory? That's somewhat unusual, unless this is one enormous e-mail you're composing.

Which line # is referenced in the latest memory error message? Also, try outputting [man]memory_get_usage/man at various places and see which lines use large amounts of memory.

slightlyeskew

Actually that number is off by a factor of 10; "numberofUber" should be around 100.

slightlyeskew

Nod, the email is pretty long. Transaction History log of stock reports. The line # referenced here was basically the same spot as before. I'll try the memory_get_usage() and get back to you.

bradgrafelman

Is it the same line #, or around the same line #? If so, can you show us which lines that corresponds to? In your first post, the line # was 98, yet the code you posted didn't even have 98 lines (thus I have no idea where "line #98" might be :p).

slightlyeskew

haha, sorry my bad I thought I had changed that. The line in my code corresponded to the line which began:

$html .= '<td>'.$numOfShares.'</td><td>...

Sorry for the confusion!

bradgrafelman

Ouch.. so it actually is the line where you're building the body of the e-mail. I'd hate to be the one who receives these e-mails.

Can you show us the memory usage of the script just before it enters that mail while() loop to build the body of the message? It's looking like it's simply the large amount of data that's causing you to bump into the memory limit, in which case I can only think of a couple of solutions:

Use [man]ini_set/man to increase the memory limit for this script (you could do this dynamically, e.g. check the memory usage to see if it's using >= 85% of the limit and, if so, increase the limit by 1.2 or something of that nature... be careful not to let it grow too large, though).
Break the data down into chunks and send multiple e-mails.
Find some other way to communicate this data :p You could instead e-mail a link to a page that outputs all of the data, rather than trying to fit it all into an e-mail.