diff options
| author | Ilia Alshanetsky <iliaa@php.net> | 2008-12-03 01:04:13 +0000 |
|---|---|---|
| committer | Ilia Alshanetsky <iliaa@php.net> | 2008-12-03 01:04:13 +0000 |
| commit | a3a73148aa55547d809fd8ce3be115c08d66e395 (patch) | |
| tree | b9093c487fc986f1309424ea40c13c2dd65d14bb /ext/imap/php_imap.c | |
| parent | fafa7f270974fe79698fc1042fac91d57cea38e0 (diff) | |
| download | php-git-a3a73148aa55547d809fd8ce3be115c08d66e395.tar.gz | |
Fixed bug #46731 (Missing validation for the options parameter of the
imap_fetch_overview() function).
Diffstat (limited to 'ext/imap/php_imap.c')
| -rw-r--r-- | ext/imap/php_imap.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c index 540de3562a..8668ffb057 100644 --- a/ext/imap/php_imap.c +++ b/ext/imap/php_imap.c @@ -3174,11 +3174,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); |