Welcome! Log In Create A New Profile

Advanced

[nginx] Fixed ngx_parse_size() / ngx_parse_offset() with 0-length strings.

Maxim Dounin
February 17, 2017 10:28AM
details: http://hg.nginx.org/nginx/rev/87cf6ddb41c2
branches:
changeset: 6917:87cf6ddb41c2
user: Maxim Dounin <mdounin@mdounin.ru>
date: Fri Feb 17 17:01:27 2017 +0300
description:
Fixed ngx_parse_size() / ngx_parse_offset() with 0-length strings.

diffstat:

src/core/ngx_parse.c | 10 ++++++++++
1 files changed, 10 insertions(+), 0 deletions(-)

diffs (27 lines):

diff --git a/src/core/ngx_parse.c b/src/core/ngx_parse.c
--- a/src/core/ngx_parse.c
+++ b/src/core/ngx_parse.c
@@ -17,6 +17,11 @@ ngx_parse_size(ngx_str_t *line)
ssize_t size, scale, max;

len = line->len;
+
+ if (len == 0) {
+ return NGX_ERROR;
+ }
+
unit = line->data[len - 1];

switch (unit) {
@@ -58,6 +63,11 @@ ngx_parse_offset(ngx_str_t *line)
size_t len;

len = line->len;
+
+ if (len == 0) {
+ return NGX_ERROR;
+ }
+
unit = line->data[len - 1];

switch (unit) {
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[nginx] Fixed ngx_parse_size() / ngx_parse_offset() with 0-length strings.

Maxim Dounin 463 February 17, 2017 10:28AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 310
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready