Update dblib_driver.c #308

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants
@jcwatson11

Bugfix for issue 63638. The patch from 1 to NULL is reverted with this commit.

Please see: https://bugs.php.net/bug.php?id=63638

This pull request addresses that issue and should be merged.

NOTE: This pull request is for the php5.4.13 branch. The original (#306) was for the master branch.

Update dblib_driver.c
Bugfix for issue 63638. The patch from 1 to NULL is reverted with this commit.
@laruence

This comment has been minimized.

Show comment Hide comment
@laruence

laruence Mar 21, 2013

Member

not sure about this fix, since the "NULL" aims to fix #63258 , needs @ssufficool review this.

Member

laruence commented Mar 21, 2013

not sure about this fix, since the "NULL" aims to fix #63258 , needs @ssufficool review this.

@fmarquis

This comment has been minimized.

Show comment Hide comment
@fmarquis

fmarquis Apr 4, 2013

ssufficool said about this bug :

I'm not sure why I used 1 instead of null for the parameter value. I have not posted patches since the source was moved to git.
Please merge this change for me if possible.

see https://bugs.php.net/bug.php?id=63258#1349972494

fmarquis commented Apr 4, 2013

ssufficool said about this bug :

I'm not sure why I used 1 instead of null for the parameter value. I have not posted patches since the source was moved to git.
Please merge this change for me if possible.

see https://bugs.php.net/bug.php?id=63258#1349972494

@WMeldon

This comment has been minimized.

Show comment Hide comment
@WMeldon

WMeldon Apr 5, 2013

+1 to see this resolved.

WMeldon commented Apr 5, 2013

+1 to see this resolved.

@php-pulls

This comment has been minimized.

Show comment Hide comment
@php-pulls

php-pulls Jun 17, 2013

Comment on behalf of stas at php.net:

looks like this one already merged

Comment on behalf of stas at php.net:

looks like this one already merged

@php-pulls php-pulls closed this Jun 17, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment