- Notifications
You must be signed in to change notification settings - Fork5
Commit89b3c6c
committed
Fix one-byte buffer overrun in contrib/test_parser.
The original coding examined the next character before verifying thatthere *is* a next character. In the worst case with the input bufferright up against the end of memory, this would result in a segfault.Problem spotted by Paul Guyot; this commit extends his patch to fix anadditional case. In addition, make the code a tad more readable by notoverloading the usage of *tlen.1 parent743ed08 commit89b3c6c
1 file changed
+10
-9
lines changedLines changed: 10 additions & 9 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
73 | 73 |
| |
74 | 74 |
| |
75 | 75 |
| |
| 76 | + | |
76 | 77 |
| |
77 | 78 |
| |
78 |
| - | |
79 | 79 |
| |
80 | 80 |
| |
81 |
| - | |
| 81 | + | |
| 82 | + | |
82 | 83 |
| |
83 | 84 |
| |
84 | 85 |
| |
85 |
| - | |
86 |
| - | |
87 |
| - | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
88 | 89 |
| |
89 | 90 |
| |
90 | 91 |
| |
91 | 92 |
| |
92 | 93 |
| |
93 | 94 |
| |
94 |
| - | |
95 |
| - | |
96 |
| - | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
97 | 98 |
| |
98 | 99 |
| |
99 | 100 |
| |
100 |
| - | |
| 101 | + | |
101 | 102 |
| |
102 | 103 |
| |
103 | 104 |
| |
|
0 commit comments
Comments
(0)