From: Ilia Alshanetsky <iliaa@php.net>
Date: Wed, 3 Dec 2008 01:04:33 +0000 (+0000)
Subject: MFB: Fixed bug #46731 (Missing validation for the options parameter of the
X-Git-Tag: BEFORE_HEAD_NS_CHANGES_MERGE~17
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6935faaa362852c58acc3f20a611b5419e8e5965;p=php

MFB: Fixed bug #46731 (Missing validation for the options parameter of the
  imap_fetch_overview() function).
---

diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c
index 67d0fedf04..b43a8bb2e6 100644
--- a/ext/imap/php_imap.c
+++ b/ext/imap/php_imap.c
@@ -3159,11 +3159,16 @@ PHP_FUNCTION(imap_fetch_overview)
 		return;
 	}
 
+	if (flags && !(flags & FT_UID)) {
+		php_error_docref(NULL TSRMLS_CC, E_WARNING, "invalid value for the options parameter");
+		RETURN_FALSE;
+	}
+
 	ZEND_FETCH_RESOURCE(imap_le_struct, pils *, &streamind, -1, "imap", le_imap);
 
 	array_init(return_value);
-	
-	status = (flags & FT_UID) 
+
+	status = (flags & FT_UID)
 		? mail_uid_sequence(imap_le_struct->imap_stream, sequence)
 		: mail_sequence(imap_le_struct->imap_stream, sequence);