From: Ilia Alshanetsky <iliaa@php.net>
Date: Tue, 1 Apr 2008 18:28:45 +0000 (+0000)
Subject: Fixed bug #44594 (imap_open() does not validate # of retries parameter)
X-Git-Tag: BEFORE_NEW_PARAMETER_PARSE~443
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7f9d35ebf79d32d59124eb6155d6d5512cf892e6;p=php

Fixed bug #44594 (imap_open() does not validate # of retries parameter)
---

diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c
index a7c554083c..b39f859932 100644
--- a/ext/imap/php_imap.c
+++ b/ext/imap/php_imap.c
@@ -798,7 +798,11 @@ static void php_imap_do_open(INTERNAL_FUNCTION_PARAMETERS, int persistent)
 #ifdef SET_MAXLOGINTRIALS
 	if (myargc == 5) {
 		convert_to_long_ex(retries);
-		mail_parameters(NIL, SET_MAXLOGINTRIALS, (void *) Z_LVAL_PP(retries));
+		if (retries < 0) {
+			php_error_docref(NULL TSRMLS_CC, E_WARNING ,"Retries must be greater or eqaul to 0");
+		} else {
+			mail_parameters(NIL, SET_MAXLOGINTRIALS, (void *) Z_LVAL_PP(retries));
+		}
 	}
 #endif