ext/ftp: Use size_t type instead of int type

Girgias · Girgias · commit 96cf1b5a9f39 · 2025-04-03T22:47:13.000+01:00
diff --git a/ext/ftp/ftp.c b/ext/ftp/ftp.c
@@ -573,12 +573,12 @@ bool ftp_rmdir(ftpbuf_t *ftp, const char *dir, const size_t dir_len)
 	return true;
 }
 
-bool ftp_chmod(ftpbuf_t *ftp, const int mode, const char *filename, const int filename_len)
+bool ftp_chmod(ftpbuf_t *ftp, const int mode, const char *filename, const size_t filename_len)
 {
 	char *buffer;
 	size_t buffer_len;
 
-	if (ftp == NULL || filename_len <= 0) {
+	if (ftp == NULL || filename_len == 0) {
 		return false;
 	}
 
diff --git a/ext/ftp/ftp.h b/ext/ftp/ftp.h
@@ -138,7 +138,7 @@ zend_string* ftp_mkdir(ftpbuf_t *ftp, const char *dir, const size_t dir_len);
 bool ftp_rmdir(ftpbuf_t *ftp, const char *dir, const size_t dir_len);
 
 /* Set permissions on a file */
-bool ftp_chmod(ftpbuf_t *ftp, const int mode, const char *filename, const int filename_len);
+bool ftp_chmod(ftpbuf_t *ftp, const int mode, const char *filename, const size_t filename_len);
 
 /* Allocate space on remote server with ALLO command
  * Many servers will respond with 202 Allocation not necessary,

Original file line number	Diff line number	Diff line change
`@@ -573,12 +573,12 @@ bool ftp_rmdir(ftpbuf_t ftp, const char dir, const size_t dir_len)`
`573`	`573`	`return true;`
`574`	`574`	`}`
`575`	`575`
`576`		`-bool ftp_chmod(ftpbuf_t ftp, const int mode, const char filename, const int filename_len)`
	`576`	`+bool ftp_chmod(ftpbuf_t ftp, const int mode, const char filename, const size_t filename_len)`
`577`	`577`	`{`
`578`	`578`	`char *buffer;`
`579`	`579`	`size_t buffer_len;`
`580`	`580`
`581`		`- if (ftp == NULL \|\| filename_len <= 0) {`
	`581`	`+ if (ftp == NULL \|\| filename_len == 0) {`
`582`	`582`	`return false;`
`583`	`583`	`}`
`584`	`584`