« first day (3938 days earlier)      last day (1004 days later) » 
00:00 - 23:0023:00 - 00:00

11:00 PM
There must be an access to this struct somewhere
Did you modify the struct?
 
I'm really sorry but I have to sleep, I have to get up at 6 (6hrs)
 
thanks, Dave, goodnight
@Girgias yes, as you can see here github.com/php/php-src/compare/…
 
no worries, ftr I am never sure whether I am talking sense here so apologies if that turns out not to be the case
 
@Dharman You didn't modify the stubs file
 
the what?
 
11:02 PM
So I reckon the class definition still has a method pointer to something which doesn't exist
You're removing a method from a Class, correct?
 
there are no classes in C
 
There should be a file ending in .stubs which is a PHP file which contains the class/function definition of said Class
I know that...
I'm talking about a PHP Class
 
I am removing an internal function that is not used anywhere in PHP source
I grepped for the name and removed it from C and H files
I see no other reference to it
 
They might be referenced by being passed as a function pointer somewhere
And if that function pointer is used somewhere else it'll bork
But are you sure it's just and internal function? And not something part of the MySQLnd hooks?
Frankly I don't know much about MySQLnd, I usually don't touch nor look at it because I have no fucking clue how the MySQL protocol works
 
I am not experienced with mysqlnd yet, but I am trying to learn. I can only be sure as far as the tools that I use don't point to this function in any way codecov.io/gh/php/php-src/src/master/ext/mysqlnd/…
 
11:12 PM
Have you tried restoring the "method" and just removing mysqlnd_field_type_name() ?
 
so obviously I am not here to say this because I am in bed, but what is on mysqlnd_ps.c:675 ?
 
Because that doesn't seem to be referenced in the method looking at your branch
 
that's where the bad ptr deref happened, so reverse engineering from that line of code should reveal the problem
/me actually sleeps
 
It does to me
The free buffer function is probably trying to free the method
 
11:19 PM
Thanks both of you, you pointed me on the right track
 
11:34 PM
@Dharman lol that feels broken /cc @Girgias :-P that is massively non-obvious even with the right tooling
 
yeah well I kinda expected something like that
 
@Dharman yw, though main thing I want you to take away is how we got there rather than what it was :-P
I have never touched mysqlnd because I had lost interest in mysql by the time I understood src, I know libpq inside tho ftr :-P
 
twitter.com/StackStatus/status/1420517891246460932 - see you all on the other side.
 
that's my cue to actually bail instead of saying I will and just opening twitter
nn x
 
11:54 PM
@DaveRandom the internet also has cats
 
I'd love to pretend I didn't see this, but honestly I am still sitting at my desk alternately playing Star Trek Fleet Command and, yeh, basically looking at cats
 
There are far worse things on the Internet.
 
since we are all here, donate to the RNLI if you are able rnli.org/support-us/give-money/donate
 
00:00 - 23:0023:00 - 00:00

« first day (3938 days earlier)      last day (1004 days later) »