sapi_lsapi_ub_write does not return bytes written in lsapi mode

Offen Anfängerfreundlich

#23,424 erstellt am 23.8.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 1/5
Geschätzter Aufwand: Unter einer Stunde
Anfängerfreundlichkeit: 90/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Aktiv
Tech-Stack: c, php
Bereich: backend

Rechercherichtung

Öffne sapi/litespeed/lsapi_main.c und untersuche sapi_lsapi_ub_write. Überprüfe anschließend den Rückgabevertrag von LSAPI_Write. Die Korrektur ist vollständig, wenn der lsapi-Modus bei einem kurzen Schreibvorgang die tatsächlich geschriebene Anzahl an Bytes meldet, während die bestehende Abort-Behandlung intakt bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Bug Status: Needs Triage

Description

sapi/litespeed/lsapi_main.c:

static size_t sapi_lsapi_ub_write(const char *str, size_t str_length)
{
    int ret;
    int remain;
    if ( lsapi_mode ) {
        ret  = LSAPI_Write( str, str_length );
        if ( ret < str_length ) {
            php_handle_aborted_connection();
            return str_length - ret;
        }
    } else {
        remain = str_length;
        while( remain > 0 ) {
            ret = write( 1, str, remain );
            if ( ret <= 0 ) {
                php_handle_aborted_connection();
                return str_length - remain;
            }
            str += ret;
            remain -= ret;
        }
    }
    return str_length;
}

It seems to me that LSAPI_Write returns the number of bytes written. If this is less than the length of the string, something went wrong. But it then returns str_length - ret, where it should return ret.

PHP Version

master

Operating System

No response

Vorherrschende Sprache: C
Sterne: 40.4k
Forks: 8.2k
Ø Merge: 2 T. 14 Std.
Gemergte PRs (30 T.): 112

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus php/php-src

Built-in server leaks a file descriptor on every HEAD request for a static file Offen

Bug SAPI: cli_server Status: Verified

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100

php/php-src#23764 · Erstellt am 19.9.2026
PDO_Firebird returns null for non-null empty BLOBs Offen

Bug Status: Needs Triage

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100

php/php-src#23758 · Erstellt am 18.9.2026
php_embed_ub_write does not return on error Offen

Bug Status: Needs Triage

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

php/php-src#23423 · Erstellt am 23.8.2026
Flaky hrtime.phpt test Offen

Bug Category: Tests Status: Verified

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

php/php-src#22508 · Erstellt am 29.6.2026
Use of PIDFile= in php-fpm.service Offen

Bug SAPI: fpm Status: Needs Triage

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

php/php-src#21740 · 1 Reaktion · Erstellt am 13.4.2026

Alle Issues in php/php-src