diff options
| author | Guido van Rossum <guido@python.org> | 1992-09-03 20:35:01 +0000 |
|---|---|---|
| committer | Guido van Rossum <guido@python.org> | 1992-09-03 20:35:01 +0000 |
| commit | 286c78c7d4e9665791bc495d89e69798a84f0601 (patch) | |
| tree | b3db73374337c96b4ae34dd541e297ea50ce71c5 /Modules/regexmodule.c | |
| parent | 8cb44e4dc9d167f75c1e086dacd5777884536bc7 (diff) | |
| download | cpython-286c78c7d4e9665791bc495d89e69798a84f0601.tar.gz | |
Improve check for offset out of range
Diffstat (limited to 'Modules/regexmodule.c')
| -rw-r--r-- | Modules/regexmodule.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/Modules/regexmodule.c b/Modules/regexmodule.c index 77daa77b6c..6afe3b7e06 100644 --- a/Modules/regexmodule.c +++ b/Modules/regexmodule.c @@ -98,6 +98,10 @@ reg_match(re, args) err_clear(); if (!getargs(args, "(s#i)", &buffer, &size, &offset)) return NULL; + if (offset < 0 || offset > size) { + err_setstr(RegexError, "match offset out of range"); + return NULL; + } } re->re_regs_valid = 0; result = re_match(&re->re_patbuf, buffer, size, offset, &re->re_regs); @@ -128,10 +132,10 @@ reg_search(re, args) err_clear(); if (!getargs(args, "(s#i)", &buffer, &size, &offset)) return NULL; - } - if (offset < 0 || offset > size) { - err_setstr(RegexError, "search offset out of range"); - return NULL; + if (offset < 0 || offset > size) { + err_setstr(RegexError, "search offset out of range"); + return NULL; + } } /* NB: In Emacs 18.57, the documentation for re_search[_2] and the implementation don't match: the documentation states that |