php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #58422 fetch(), getAccessToken() - Memory leaks
Submitted: 2008-11-21 07:40 UTC Modified: 2008-11-22 15:43 UTC
From: felipe@php.net Assigned: jawed (profile)
Status: Closed Package: oauth (PECL)
PHP Version: 5_2 CVS-2008-11-21 (dev) OS: Linux
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
34 - 8 = ?
Subscribe to this entry?

 
 [2008-11-21 07:40 UTC] felipe@php.net 
Description:
------------
The following code in oauth::fetch needs be moved to down of its arguments checks, because it's causing memory leak obviously:

ALLOC_HASHTABLE(args);
zend_hash_init(args, 0, NULL, ZVAL_PTR_DTOR, 0);


Reproduce code:
---------------
<?php

$x = new oauth(1,1);
$x->fetch();

Actual result:
--------------
Warning: Wrong parameter count for OAuth::fetch() in /home/felipe/dev/bug.php on line 4
[Fri Nov 21 10:33:41 2008]  Script:  '../bug.php'
/home/felipe/dev/php5_2/ext/oauth/oauth.c(1021) :  Freeing 0x085FAEC8 (44 bytes), script=../bug.php
[Fri Nov 21 10:33:41 2008]  Script:  '../bug.php'
/home/felipe/dev/php5_2/ext/oauth/oauth.c(1022) :  Freeing 0x085FAF24 (32 bytes), script=../bug.php
/home/felipe/dev/php5_2/Zend/zend_alloc.c(2386) : Actual location (location was relayed)
=== Total 2 memory leaks detected ===

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2008-11-21 07:43 UTC] felipe@php.net 
This also occurs with:
<?php

$x = new oauth(1,1);
$x->getAccessToken();

Warning: Wrong parameter count for OAuth::getAccessToken() in /home/felipe/dev/bug.php on line 4
[Fri Nov 21 10:42:32 2008]  Script:  '../bug.php'
/home/felipe/dev/php5_2/ext/oauth/oauth.c(1104) :  Freeing 0x085FAF08 (44 bytes), script=../bug.php
[Fri Nov 21 10:42:32 2008]  Script:  '../bug.php'
/home/felipe/dev/php5_2/ext/oauth/oauth.c(1105) :  Freeing 0x085FAF64 (32 bytes), script=../bug.php
/home/felipe/dev/php5_2/Zend/zend_alloc.c(2386) : Actual location (location was relayed)
=== Total 2 memory leaks detected ===
 [2008-11-21 08:05 UTC] felipe@php.net 
The patch for Bug #15110 includes the fix for this bug too.
 [2008-11-22 04:49 UTC] jawed@php.net 
Please check and verify whether or not the latest source in CVS resolves this bug.
 [2008-11-22 15:43 UTC] felipe@php.net 
Yes, it's ok now.
 
PHP Copyright © 2001-2023 The PHP Group
All rights reserved. 		Last updated: Mon Sep 25 04:01:25 2023 UTC