diff options
| author | Anatol Belski <ab@php.net> | 2017-07-23 17:33:18 +0200 |
|---|---|---|
| committer | Anatol Belski <ab@php.net> | 2017-07-23 17:36:10 +0200 |
| commit | 6809be20902bf83a9d21e9658b07d3365557ffe6 (patch) | |
| tree | 75330a675ddf6f572a3aa5b5984af5f3b48f58e6 | |
| parent | 7496bad2ace2be2c0d967f7afefd6e9c8e96af79 (diff) | |
| download | php-git-6809be20902bf83a9d21e9658b07d3365557ffe6.tar.gz | |
fix warnings and datatype
ident
| -rw-r--r-- | ext/mbstring/mbstring.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/ext/mbstring/mbstring.c b/ext/mbstring/mbstring.c index 9f96bc9c72..561794cbb6 100644 --- a/ext/mbstring/mbstring.c +++ b/ext/mbstring/mbstring.c @@ -4971,8 +4971,7 @@ static inline zend_long php_mb_ord(const char* str, size_t str_len, const char* const mbfl_encoding *enc; enum mbfl_no_encoding no_enc; char* ret; - size_t ret_len; - unsigned char char_len; + size_t ret_len, char_len; zend_long cp; enc = php_mb_get_encoding(enc_name); @@ -5429,16 +5428,16 @@ MBSTRING_API size_t php_mb_stripos(int mode, const char *old_haystack, size_t ol size_t haystack_char_len = mbfl_strlen(&haystack); if (mode) { - if ((offset > 0 && offset > haystack_char_len) || - (offset < 0 && -offset > haystack_char_len)) { + if ((offset > 0 && (size_t)offset > haystack_char_len) || + (offset < 0 && (size_t)(-offset) > haystack_char_len)) { php_error_docref(NULL, E_WARNING, "Offset is greater than the length of haystack string"); break; } } else { if (offset < 0) { - offset += (long)haystack_char_len; + offset += (zend_long)haystack_char_len; } - if (offset < 0 || offset > haystack_char_len) { + if (offset < 0 || (size_t)offset > haystack_char_len) { php_error_docref(NULL, E_WARNING, "Offset not contained in string"); break; } |