Merge branch 'PHP-8.4' into PHP-8.5

iliaal · iliaal · commit b5e26537053b · 2026-06-21T17:18:53.000-04:00
* PHP-8.4:
  ext/ftp: fix off-by-one terminator write in ftp_readline()
diff --git a/ext/ftp/ftp.c b/ext/ftp/ftp.c
@@ -1254,7 +1254,7 @@ static bool ftp_readline(ftpbuf_t *ftp)
 		}
 
 		data = eol;
-		if ((rcvd = my_recv(ftp, ftp->fd, data, size)) < 1) {
+		if (size < 2 || (rcvd = my_recv(ftp, ftp->fd, data, size - 1)) < 1) {
 			*data = 0;
 			return false;
 		}
diff --git a/ext/ftp/tests/bug80901.phpt b/ext/ftp/tests/bug80901.phpt
@@ -16,4 +16,4 @@ ftp_systype($ftp);
 --EXPECTF--
 bool(true)
 
-Warning: ftp_systype(): **************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************** in %s on line %d
+Warning: ftp_systype(): *************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************** in %s on line %d

Original file line number	Diff line number	Diff line change
`@@ -1254,7 +1254,7 @@ static bool ftp_readline(ftpbuf_t *ftp)`
`1254`	`1254`	`}`
`1255`	`1255`
`1256`	`1256`	`data = eol;`
`1257`		`- if ((rcvd = my_recv(ftp, ftp->fd, data, size)) < 1) {`
	`1257`	`+ if (size < 2 \|\| (rcvd = my_recv(ftp, ftp->fd, data, size - 1)) < 1) {`
`1258`	`1258`	`*data = 0;`
`1259`	`1259`	`return false;`
`1260`	`1260`	`}`