return false after determining that a message number is bad, instead of

going on to crash.

diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c
index e06bc7b5d793f92491455de0cd45ba33cbdc04f6..6cf20e420b2725609398b0133e23830e64de59c4 100644 (file)
--- a/ext/imap/php_imap.c
+++ b/ext/imap/php_imap.c
@@ -1566,7 +1566,9 @@ PHP_FUNCTION(imap_headerinfo)
        
        if (!msgno->value.lval || msgno->value.lval < 1 || (unsigned) msgno->value.lval > imap_le_struct->imap_stream->nmsgs) {
                php_error(E_WARNING, "Bad message number");
+               RETURN_FALSE;
        }
+       
        if (mail_fetchstructure(imap_le_struct->imap_stream, msgno->value.lval, NIL)) {
                cache = mail_elt(imap_le_struct->imap_stream, msgno->value.lval);
        } else {