Skip to content

Commit a78769a

Browse files
datibbawdatibbaw
committed
Few more fixes
- Fixed crash - Updated test cases due to changes in RegexIterator::accept()
1 parent 3c90b64 commit a78769a

File tree

3 files changed

+1
-215
lines changed

3 files changed

+1
-215
lines changed

‎ext/spl/spl_iterators.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2268,14 +2268,13 @@ SPL_METHOD(RecursiveRegexIterator, getChildren)
22682268
if (!EG(exception)) {
22692269
zval args[5];
22702270

2271-
args[0] = retval;
2271+
ZVAL_COPY(&args[0], &retval);
22722272
ZVAL_STR_COPY(&args[1], intern->u.regex.regex);
22732273
ZVAL_LONG(&args[2], intern->u.regex.mode);
22742274
ZVAL_LONG(&args[3], intern->u.regex.flags);
22752275
ZVAL_LONG(&args[4], intern->u.regex.preg_flags);
22762276

22772277
spl_instantiate_arg_n(Z_OBJCE_P(getThis()), return_value, 5, args TSRMLS_CC);
2278-
zval_ptr_dtor(&args[1]);
22792278
}
22802279
zval_ptr_dtor(&retval);
22812280
} /* }}} */

‎ext/spl/tests/iterator_052.phpt

Lines changed: 0 additions & 117 deletions
Original file line numberDiff line numberDiff line change
@@ -46,18 +46,6 @@ var_dump($ar);
4646
<?php exit(0); ?>
4747
--EXPECTF--
4848
bool(true)
49-
int(0)
50-
array(3) {
51-
[0]=>
52-
array(0) {
53-
}
54-
[1]=>
55-
array(0) {
56-
}
57-
[2]=>
58-
array(0) {
59-
}
60-
}
6149
bool(true)
6250
int(1)
6351
array(3) {
@@ -97,71 +85,11 @@ array(3) {
9785
}
9886
}
9987
bool(true)
100-
int(3)
101-
array(3) {
102-
[0]=>
103-
array(0) {
104-
}
105-
[1]=>
106-
array(0) {
107-
}
108-
[2]=>
109-
array(0) {
110-
}
111-
}
11288
bool(true)
113-
int(4)
114-
array(3) {
115-
[0]=>
116-
array(0) {
117-
}
118-
[1]=>
119-
array(0) {
120-
}
121-
[2]=>
122-
array(0) {
123-
}
124-
}
12589
bool(false)
12690
bool(true)
127-
int(6)
128-
array(3) {
129-
[0]=>
130-
array(0) {
131-
}
132-
[1]=>
133-
array(0) {
134-
}
135-
[2]=>
136-
array(0) {
137-
}
138-
}
13991
bool(true)
140-
int(7)
141-
array(3) {
142-
[0]=>
143-
array(0) {
144-
}
145-
[1]=>
146-
array(0) {
147-
}
148-
[2]=>
149-
array(0) {
150-
}
151-
}
15292
bool(true)
153-
int(8)
154-
array(3) {
155-
[0]=>
156-
array(0) {
157-
}
158-
[1]=>
159-
array(0) {
160-
}
161-
[2]=>
162-
array(0) {
163-
}
164-
}
16593
bool(true)
16694
int(0)
16795
array(2) {
@@ -217,56 +145,11 @@ array(2) {
217145
}
218146
}
219147
bool(true)
220-
int(3)
221-
array(2) {
222-
[0]=>
223-
array(0) {
224-
}
225-
[1]=>
226-
array(0) {
227-
}
228-
}
229148
bool(true)
230-
int(4)
231-
array(2) {
232-
[0]=>
233-
array(0) {
234-
}
235-
[1]=>
236-
array(0) {
237-
}
238-
}
239149
bool(false)
240150
bool(true)
241-
int(6)
242-
array(2) {
243-
[0]=>
244-
array(0) {
245-
}
246-
[1]=>
247-
array(0) {
248-
}
249-
}
250151
bool(true)
251-
int(7)
252-
array(2) {
253-
[0]=>
254-
array(0) {
255-
}
256-
[1]=>
257-
array(0) {
258-
}
259-
}
260152
bool(true)
261-
int(8)
262-
array(2) {
263-
[0]=>
264-
array(0) {
265-
}
266-
[1]=>
267-
array(0) {
268-
}
269-
}
270153
object(ArrayIterator)#%d (1) {
271154
["storage":"ArrayIterator":private]=>
272155
array(9) {

‎ext/spl/tests/iterator_053.phpt

Lines changed: 0 additions & 96 deletions
Original file line numberDiff line numberDiff line change
@@ -46,110 +46,14 @@ var_dump($ar);
4646
<?php exit(0); ?>
4747
--EXPECTF--
4848
bool(true)
49-
int(0)
50-
array(3) {
51-
[0]=>
52-
array(0) {
53-
}
54-
[1]=>
55-
array(0) {
56-
}
57-
[2]=>
58-
array(0) {
59-
}
60-
}
6149
bool(true)
62-
int(1)
63-
array(3) {
64-
[0]=>
65-
array(0) {
66-
}
67-
[1]=>
68-
array(0) {
69-
}
70-
[2]=>
71-
array(0) {
72-
}
73-
}
7450
bool(true)
75-
int(2)
76-
array(3) {
77-
[0]=>
78-
array(0) {
79-
}
80-
[1]=>
81-
array(0) {
82-
}
83-
[2]=>
84-
array(0) {
85-
}
86-
}
8751
bool(true)
88-
int(3)
89-
array(3) {
90-
[0]=>
91-
array(0) {
92-
}
93-
[1]=>
94-
array(0) {
95-
}
96-
[2]=>
97-
array(0) {
98-
}
99-
}
10052
bool(true)
101-
int(4)
102-
array(3) {
103-
[0]=>
104-
array(0) {
105-
}
106-
[1]=>
107-
array(0) {
108-
}
109-
[2]=>
110-
array(0) {
111-
}
112-
}
11353
bool(false)
11454
bool(true)
115-
int(6)
116-
array(3) {
117-
[0]=>
118-
array(0) {
119-
}
120-
[1]=>
121-
array(0) {
122-
}
123-
[2]=>
124-
array(0) {
125-
}
126-
}
12755
bool(true)
128-
int(7)
129-
array(3) {
130-
[0]=>
131-
array(0) {
132-
}
133-
[1]=>
134-
array(0) {
135-
}
136-
[2]=>
137-
array(0) {
138-
}
139-
}
14056
bool(true)
141-
int(8)
142-
array(3) {
143-
[0]=>
144-
array(0) {
145-
}
146-
[1]=>
147-
array(0) {
148-
}
149-
[2]=>
150-
array(0) {
151-
}
152-
}
15357
bool(true)
15458
int(0)
15559
array(2) {

0 commit comments

Comments
 (0)