-
jfqd1
jperkin: With the latest php 8.2 update in trunk we get "Memory fault(coredump)" crashes when running php on the commandline. Nextcloud for sample has a cron script, which now fails sometimes but not every time. Strange is, that there is no coredump. Any idea what changed in the latest php 8.2 update?
-
jperkin
no, can you get a reproducer for me? core dumps will be wherever coreadm has them configured
-
jfqd1
jperkin: Sorry, what do you meen be reproducer? For Nextcloud the cron script is like: /opt/local/bin/php /var/www/htdocs/nextcloud/current/cron.php
-
jperkin
a way to reproduce the crash
-
jfqd1
We are using the regular coreadm config, no changes.
-
jperkin
I don't have any php cores on my build hosts, so it's not something that is triggered by any of the php package builds at least
-
jperkin
so it says coredump but there's no core in the coreadm directory?
-
jfqd1
jperkin: reproducing will be difficult, cause it not always crashes only sometimes. I will first create a new image with php 8.3 and will see if the issue goes away. Yes it says coredump, but no core file in /var/cores
-
jperkin
/var/cores isn't the default?
-
jfqd1
Ah I see there are core files in /zones/<uuid>/cores. OK, will investigate and come back :-)
-
jperkin
ah, good, so pstack on one of them will at least give some basic clues as to what to look for
-
jfqd1
jperkin: Crashes in fffffc7feefe895d iconv (1889b50, fffffc7fffdfda78, fffffc7fffdfda80, fffffc7fffdfda90, fffffc7fffdfda88) + 2d
-
jfqd1
So it might be more an issue of iconv?
-
jperkin
can you gist the full stack somewhere? as a workaround you may find LANG=C or whatever helps
-
jperkin
the only thing I can think of that might be related is the recent icu update
-
jfqd1
-
jfqd1
And I added LANG=C and will see what is happening :-)
-
jfqd1
Thx a lot for your help!
-
jperkin
I'll dig around the icu/iconv/php-iconv test suites and see if they trigger anything
-
jperkin
ok, good news is that php-iconv tests also fail and dump core, so I have plenty to go on ;)
-
jperkin
ok found the problem, I'll get that in the next build
-
jfqd1
jperkin: Thx for digging into it! Be happy to test if the next trunk updates are ready :-)
-
jperkin
I've merged it early so will let you know when available