forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commite060cd5
committed
Avoid copying undefined data in _readA_Const().
nodeRead() will have created a Node struct that's only allocated bigenough for the specific node type, so copying sizeof(union ValUnion)can be copying too much. This provokes valgrind complaints, and withvery bad luck could perhaps result in SIGSEGV.While at it, tidy up _equalA_Const to avoid duplicate checks of isnull.Per report from Alexander Lakhin. This code is new as ofa6bc330,so no need to back-patch.Discussion:https://postgr.es/m/4995256b-cc65-170e-0b22-60ad2cd535f1@gmail.com1 parent4f46f87 commite060cd5
2 files changed
+27
-6
lines changedLines changed: 3 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
133 | 133 |
| |
134 | 134 |
| |
135 | 135 |
| |
136 |
| - | |
137 |
| - | |
138 |
| - | |
139 |
| - | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
140 | 139 |
| |
141 | 140 |
| |
142 |
| - | |
143 | 141 |
| |
144 | 142 |
| |
145 | 143 |
| |
|
Lines changed: 24 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
305 | 305 |
| |
306 | 306 |
| |
307 | 307 |
| |
| 308 | + | |
308 | 309 |
| |
309 | 310 |
| |
310 | 311 |
| |
311 | 312 |
| |
312 | 313 |
| |
313 | 314 |
| |
314 | 315 |
| |
315 |
| - | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
316 | 339 |
| |
317 | 340 |
| |
318 | 341 |
| |
|
0 commit comments
Comments
(0)