php / php/php-src

Timer leaks on pcntl_fork in ZTS build

Ouverte
#12,298 8 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Bug Extension: pcntl Status: Needs Triage
Langage dominant
C
Étoiles
40.4k
Forks
8.2k
Merge moyen
2 j 15 h
PR mergées (30 j)
103

Description

Description

The following code:

<?php
pcntl_fork();

Resulted in this output (in child process):

=================================================================
==58097==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0xffffa37f943b in malloc (/usr/lib/aarch64-linux-gnu/libasan.so.5+0xcf43b)
    #1 0xffffa3622aaf in timer_create ../sysdeps/unix/sysv/linux/timer_create.c:59
    #2 0xaaaacd663a6b in zend_max_execution_timer_init /usr/local/src/php/Zend/zend_max_execution_timer.c:46
    #3 0xaaaacd2913c7 in init_executor /usr/local/src/php/Zend/zend_execute_API.c:202
    #4 0xaaaacd2e3497 in zend_activate /usr/local/src/php/Zend/zend.c:1257
    #5 0xaaaacd0b09c3 in php_request_startup /usr/local/src/php/main/main.c:1758
    #6 0xaaaacd7b92db in do_cli /usr/local/src/php/sapi/cli/php_cli.c:934
    #7 0xaaaacd7bbd57 in main /usr/local/src/php/sapi/cli/php_cli.c:1340
    #8 0xffffa08c7da3 in __libc_start_main ../csu/libc-start.c:308
    #9 0xaaaacbea50f3  (/opt/php/debug-zts-asan/bin/php+0x5b50f3)

There's an explicit comment that this shouldn't be freed after a fork:

void zend_max_execution_timer_shutdown(void) /* {{{ */
{
	/* Don't try to delete a timer created before a call to fork() */
	if (EG(pid) != getpid()) {
		return;
	}

So paging @dunglas who wrote that comment.

PHP Version

PHP 8.3

Operating System

No response

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Reproduisez le cas PHP 8.3 ZTS avec pcntl_fork sous AddressSanitizer, puis lisez zend_max_execution_timer.c ainsi que les chemins d’initialisation et d’arrêt présentés dans le rapport. Utilisez la trace de fuite du processus enfant comme référence de départ ; le travail est terminé lorsque le reproducteur ne signale plus le temporisateur ayant fui, sans régression dans la gestion du temporisateur du processus parent.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
c, php
Domaine
backend, operating-systems
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.