forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit41bd2cf
committed
Prevent interrupts while reporting non-ERROR elog messages.
This should eliminate the risk of recursive entry to syslog(3), whichappears to be the cause of the hang reported in bug #9551 from JamesMorton.Arguably, the real problem here is auth.c's willingness to turn onImmediateInterruptOK while executing fairly wide swaths of backend code.We may well need to work at narrowing the code ranges in which theauthentication_timeout interrupt is enabled. For the moment, though,this is a cheap and reasonably noninvasive fix for a field-reportedfailure; the other approach would be complex and not necessarilybug-free itself.Back-patch to all supported branches.1 parent0d11fed commit41bd2cf
1 file changed
+25
-13
lines changedLines changed: 25 additions & 13 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
404 | 404 |
| |
405 | 405 |
| |
406 | 406 |
| |
| 407 | + | |
407 | 408 |
| |
408 | 409 |
| |
409 | 410 |
| |
410 | 411 |
| |
411 | 412 |
| |
412 | 413 |
| |
| 414 | + | |
| 415 | + | |
| 416 | + | |
| 417 | + | |
| 418 | + | |
| 419 | + | |
| 420 | + | |
| 421 | + | |
| 422 | + | |
| 423 | + | |
| 424 | + | |
| 425 | + | |
413 | 426 |
| |
414 | 427 |
| |
415 | 428 |
| |
| |||
435 | 448 |
| |
436 | 449 |
| |
437 | 450 |
| |
438 |
| - | |
439 |
| - | |
440 |
| - | |
441 |
| - | |
442 |
| - | |
443 |
| - | |
| 451 | + | |
444 | 452 |
| |
445 | 453 |
| |
446 | 454 |
| |
447 | 455 |
| |
448 | 456 |
| |
| 457 | + | |
| 458 | + | |
| 459 | + | |
| 460 | + | |
449 | 461 |
| |
450 | 462 |
| |
451 | 463 |
| |
| |||
512 | 524 |
| |
513 | 525 |
| |
514 | 526 |
| |
515 |
| - | |
516 |
| - | |
517 |
| - | |
518 |
| - | |
| 527 | + | |
519 | 528 |
| |
520 | 529 |
| |
521 | 530 |
| |
| |||
548 | 557 |
| |
549 | 558 |
| |
550 | 559 |
| |
551 |
| - | |
552 | 560 |
| |
553 | 561 |
| |
554 | 562 |
| |
555 | 563 |
| |
556 | 564 |
| |
557 | 565 |
| |
558 |
| - | |
559 |
| - | |
| 566 | + | |
| 567 | + | |
| 568 | + | |
| 569 | + | |
| 570 | + | |
| 571 | + | |
560 | 572 |
| |
561 | 573 |
| |
562 | 574 |
| |
|
0 commit comments
Comments
(0)