Conversation
ext/mysqlnd/mysqlnd_ps_codec.c
Outdated
There was a problem hiding this comment.
Accessing DVAL after casting to long? That can't be right...
There was a problem hiding this comment.
You're right nikic, I put my focus on my problem but forgetting to get everything right, sorry for that.
but the convert to double issue really caused my problem, maybe I should get another temporary zval to make old code working as well
There was a problem hiding this comment.
Die Zustellung an folgende Empfänger oder Gruppen verzögert sich:
[email protected]:[email protected]
Betreff: Re: [php-src] Fixed bug #66124 (#532)
Diese Nachricht wurde noch nicht zugestellt. Es wird weiterhin versucht, die Nachricht zuzustellen.
Der Server wird noch 2 Tage, 22 Stunden und 58 Minuten versuchen, die Nachricht zuzustellen. Sie erhalten eine Benachrichtigung, falls die Nachricht bis dahin nicht übermittelt werden konnte.
There was a problem hiding this comment.
Die Zustellung an folgende Empfänger oder Gruppen verzögert sich:
[email protected]:[email protected]
Betreff: Re: [php-src] Fixed bug #66124 (#532)
Diese Nachricht wurde noch nicht zugestellt. Es wird weiterhin versucht, die Nachricht zuzustellen.
Der Server wird noch 2 Tage, 22 Stunden und 57 Minuten versuchen, die Nachricht zuzustellen. Sie erhalten eine Benachrichtigung, falls die Nachricht bis dahin nicht übermittelt werden konnte.
|
@nikic would you please review the latest commit? |
There was a problem hiding this comment.
You created the zval with MAKE_STD_ZVAL but destroy with zval_dtor, that will leak the container. You're probably looking for something like this:
zval tmp_data;
ZVAL_ZVAL(&tmp_data, (copies && copies[i]) ? copies[i]: stmt->param_bind[i].zv, 1, 0);
convert_to_double(&tmp_data);
// ...
zval_dtor(&tmp_data);
That's the semi-standard approach to performing a cast without touching the zval. The ZVAL_ZVAL is the same as ZVAL_COPY_VALUE+zval_copy_ctor.
3 participants
https://bugs.php.net/bug.php?id=66124