diff options
| author | Antony Dovgal <tony2001@php.net> | 2005-01-20 14:04:50 +0000 |
|---|---|---|
| committer | Antony Dovgal <tony2001@php.net> | 2005-01-20 14:04:50 +0000 |
| commit | 9812f552899b0fecd7f869b9760516e920712e05 (patch) | |
| tree | dd54560c3b9587a9c9c36def374c7d804a926075 /ext/imap/php_imap.c | |
| parent | a830b0fc6b2c3651f0c8ed44b83708945efaca21 (diff) | |
| download | php-git-9812f552899b0fecd7f869b9760516e920712e05.tar.gz | |
fix segfault in imap_headerinfo() when fromlength or subjectlength are less than 0
+ fix protos
Diffstat (limited to 'ext/imap/php_imap.c')
| -rw-r--r-- | ext/imap/php_imap.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c index 77d83cefba..b243fb37a1 100644 --- a/ext/imap/php_imap.c +++ b/ext/imap/php_imap.c @@ -1525,12 +1525,20 @@ PHP_FUNCTION(imap_headerinfo) convert_to_long_ex(msgno); if (myargc >= 3) { - convert_to_long_ex(fromlength); + convert_to_long_ex(fromlength); + if (Z_LVAL_PP(fromlength) < 0) { + php_error_docref(NULL TSRMLS_CC, E_WARNING, "From length has to be greater than or equal to 0"); + RETURN_FALSE; + } } else { fromlength = 0x00; } if (myargc >= 4) { convert_to_long_ex(subjectlength); + if (Z_LVAL_PP(subjectlength) < 0) { + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Subject length has to be greater than or equal to 0"); + RETURN_FALSE; + } } else { subjectlength = 0x00; } @@ -1798,7 +1806,7 @@ PHP_FUNCTION(imap_fetchstructure) } /* }}} */ -/* {{{ proto string imap_fetchbody(resource stream_id, int msg_no, int section [, int options]) +/* {{{ proto string imap_fetchbody(resource stream_id, int msg_no, string section [, int options]) Get a specific body section */ PHP_FUNCTION(imap_fetchbody) { @@ -2612,7 +2620,7 @@ PHP_FUNCTION(imap_status) } /* }}} */ -/* {{{ proto object imap_bodystruct(resource stream_id, int msg_no, int section) +/* {{{ proto object imap_bodystruct(resource stream_id, int msg_no, string section) Read the structure of a specified body section of a specific message */ PHP_FUNCTION(imap_bodystruct) { |