nginx/nginx

Potential null dereference in ngx_time.c

Geschlossen

#169 geöffnet am 12.09.2024

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C (7.903 Forks)batch import
Complexity - hardbuggood first issue

Repository-Metriken

Stars
 (30.331 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 27T 8h) (17 gemergte PRs in 30 T)

Beschreibung

In results of static analyze of nginx sources I founded potential null dereference

https://github.com/nginx/nginx/blob/00637cce366f17b78fe1ed5c1ef0e534143045f6/src/os/unix/ngx_time.c#L48-L50

localtime() function potentially can return NULL value, which will dereference in strftime(). How about to add check for this case?

Contributor Guide

Research-Richtung
Überprüfen Sie den Code unter dem bereitgestellten Link (ngx time.c Zeilen 48-50). Implementieren Sie eine Nullprüfung für den Rückgabewert von localtime(), bevor er an strftime() übergeben wird. Erwägen Sie die Verwendung von localtime r(), wenn Thread Sicherheit ein Anliegen ist. Stellen Sie sicher, dass die Korrektur keine Regression einführt.
Tech Stack
c
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Blockiert
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C programmingunderstanding of localtime()
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70