php_embed_ub_write does not return on error

Description

sapi/embed/php_embed.c:

/* SAPIs only have unbuffered write operations. This is because PHP's output
 * buffering feature will handle any buffering of the output and invoke the
 * SAPI unbuffered write operation when it flushes the buffer.
 */
static size_t php_embed_ub_write(const char *str, size_t str_length)
{
	const char *ptr = str;
	size_t remaining = str_length;
	size_t ret;

	while (remaining > 0) {
		ret = php_embed_single_write(ptr, remaining);
		if (!ret) {
			php_handle_aborted_connection();
		}
		ptr += ret;
		remaining -= ret;
	}

	return str_length;
}

php_handle_aborted_connection does not always abort, i.e. if ignore_user_abort=1. So I sending fails and ret is 0, then remaining is never decremented and this loops forever.

It should probably return str_length - remaining after php_handle_aborted_connection().

PHP Version

master

Operating System

No response

Beginne in sapi/embed/php_embed.c bei php_embed_ub_write und verfolge den Fehlerpfad von php_embed_single_write, gefolgt von php_handle_aborted_connection. Stelle sicher, dass sich ein Schreiben von null Bytes nicht unbegrenzt wiederholen kann, wenn die Verbindung nicht abgebrochen wurde. Als erledigt gilt die Aufgabe, wenn die Funktion nach dem Fehler die Anzahl der geschriebenen Bytes zurückgibt, statt endlos zu schleifen.