php / php/php-src

ext/date truncates error messages with null bytes

Ouverte
#22,993 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Bug Extension: date Status: Verified
Langage dominant
C
Étoiles
40.4k
Forks
8.1k
Merge moyen
2 j 13 h
PR mergées (30 j)
96

Description

date_throw_uninitialized_error() (C function):

<?php

$o = new class extends DateTime {
    public function __construct() {}
};
var_dump(get_class($o));
$o->format("");

OnUpdate_date_timezone ini handler:

<?php

ini_set("date.timezone", "foo\0bar");

php_date_initialize() (C function):

<?php

new DateTimeImmutable("foo\0bar");

php_date_modify() (C function):

<?php

$now = new DateTime();
$now->modify("foo\0bar");

date_interval_initialize() (C function):

<?php

new DateInterval("foo\0bar");

php_date_interval_initialize_from_hash() (C function):

<?php

var_dump(unserialize("O:12:\"DateInterval\":2:{s:11:\"from_string\";b:1;s:11:\"date_string\";s:7:\"foo\0bar\";}"));

date_interval_create_from_date_string() (C function):

<?php

date_interval_create_from_date_string("foo\0bar");

DateInterval::createFromDateString():

<?php

DateInterval::createFromDateString("foo\0bar");

date_period_initialize() (C function):

<?php

DatePeriod::createFromISO8601String("foo\0bar");

date_period_initialize() (C function):

<?php

DatePeriod::createFromISO8601String("foo\0bar");

date_default_timezone_set():

<?php

date_default_timezone_set("foo\0bar");

date_period_unset_property() (C function):

<?php

$o = new class() extends DatePeriod {
    public function __construct() {}
};
unset($o->start);

Tested with 3v4l

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

Commencez dans l’implémentation de ext/date, au niveau des fonctions C nommées, et reproduisez les cas d’octet nul listés avec les extraits PHP fournis. Suivez la manière dont chaque message d’erreur est construit, puis ajoutez une couverture pour les chemins concernés et vérifiez que les messages produits préservent l’entrée pertinente ; exécutez les tests de l’extension date pour confirmer tous les points d’entrée listés.

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

Évaluation

Stack technique
c, php
Domaine
backend
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

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