Bug #10458 Updated: Bugs #9257 and #10292 located and fixed - see diff

From: Date: Thu, 03 May 2001 07:30:52 +0000
Subject: Bug #10458 Updated: Bugs #9257 and #10292 located and fixed - see diff
Groups: php.dev 
Request: Send a blank email to php-dev+get-53522@lists.php.net to get a copy of this message
ID: 10458 Updated by: sniper Reported By: lawe@tdk.dk Old-Status: Open Status: Closed Bug Type: InterBase related PHP Version: 4.0.4pl1 Assigned To: Comments: Andi applied your patch. --Jani Previous Comments: --------------------------------------------------------------------------- [2001-04-23 10:58:00] lawe@tdk.dk Hi! See additional comments under #10292. I have located and corrected the two bugs. Here is the diff between the new ./ext/interbase.c and the old one (version 1.48): --------------- $diff interbase.c interbase.orig.c 1731d1730 < val->value.str.val[len] = 0; 1736c1735 < val->value.str.val = php_addslashes(val->value.str.val, len, &len, 1); --- > val->value.str.val = php_addslashes(val->value.str.val, len, &len, 0); 1754d1752 < 1784,1785c1782,1783 < val->value.str.len = sprintf(string_data, "%Ld.%0*Ld", < (ISC_INT64) (*((ISC_INT64 *)data) / (int) pow(10.0, (double) -scale)), -scale, --- > val->value.str.len = sprintf(string_data, "%Ld.%Ld", > (ISC_INT64) (*((ISC_INT64 *)data) / (int) pow(10.0, > (double) -scale)), $ --------------- It would be very convenient, if a modified php_interbase.dll could be made available soon. The Linux and FreeBSD users can just make the corrections and compile again. Ad #9257) The old code didn't cope with the scale parameter to adjust the width of the fractional part (zero-padded). Yours sincerely, Lars Westermann --------------------------------------------------------------------------- ATTENTION! Do NOT reply to this email! To reply, use the web interface found at http://bugs.php.net/?id=10458&edit=2

« previous php.dev (#53522) next »