forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit28e4632
committed
Centralize logic for restoring errno in signal handlers.
Presently, we rely on each individual signal handler to save theinitial value of errno and then restore it before returning ifneeded. This is easily forgotten and, if missed, often goesundetected for a long time.In commit3b00fdb, we introduced a wrapper signal handlerfunction that checks whether MyProcPid matches getpid(). Thiscommit moves the aforementioned errno restoration code from theindividual signal handlers to the new wrapper handler so that we nolonger need to worry about missing it.Reviewed-by: Andres Freund, Noah MischDiscussion:https://postgr.es/m/20231121212008.GA3742740%40nathanxps131 parent3b00fdb commit28e4632
File tree
15 files changed
+6
-87
lines changed- doc/src/sgml
- src
- backend
- postmaster
- replication
- storage/ipc
- tcop
- utils/misc
- fe_utils
- port
15 files changed
+6
-87
lines changedLines changed: 0 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1007 | 1007 |
| |
1008 | 1008 |
| |
1009 | 1009 |
| |
1010 |
| - | |
1011 |
| - | |
1012 | 1010 |
| |
1013 | 1011 |
| |
1014 |
| - | |
1015 |
| - | |
1016 | 1012 |
| |
1017 | 1013 |
| |
1018 |
| - | |
1019 |
| - | |
1020 |
| - | |
1021 |
| - | |
1022 | 1014 |
| |
1023 | 1015 |
| |
1024 | 1016 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1410 | 1410 |
| |
1411 | 1411 |
| |
1412 | 1412 |
| |
1413 |
| - | |
1414 |
| - | |
1415 | 1413 |
| |
1416 | 1414 |
| |
1417 |
| - | |
1418 |
| - | |
1419 | 1415 |
| |
1420 | 1416 |
| |
1421 | 1417 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
852 | 852 |
| |
853 | 853 |
| |
854 | 854 |
| |
855 |
| - | |
856 |
| - | |
857 | 855 |
| |
858 | 856 |
| |
859 | 857 |
| |
860 | 858 |
| |
861 | 859 |
| |
862 |
| - | |
863 |
| - | |
864 | 860 |
| |
865 | 861 |
| |
866 | 862 |
| |
|
Lines changed: 0 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
60 | 60 |
| |
61 | 61 |
| |
62 | 62 |
| |
63 |
| - | |
64 |
| - | |
65 | 63 |
| |
66 | 64 |
| |
67 |
| - | |
68 |
| - | |
69 | 65 |
| |
70 | 66 |
| |
71 | 67 |
| |
| |||
108 | 104 |
| |
109 | 105 |
| |
110 | 106 |
| |
111 |
| - | |
112 |
| - | |
113 | 107 |
| |
114 | 108 |
| |
115 |
| - | |
116 |
| - | |
117 | 109 |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
283 | 283 |
| |
284 | 284 |
| |
285 | 285 |
| |
286 |
| - | |
287 |
| - | |
288 | 286 |
| |
289 | 287 |
| |
290 | 288 |
| |
291 |
| - | |
292 |
| - | |
293 | 289 |
| |
294 | 290 |
| |
295 | 291 |
| |
|
Lines changed: 0 additions & 16 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2612 | 2612 |
| |
2613 | 2613 |
| |
2614 | 2614 |
| |
2615 |
| - | |
2616 |
| - | |
2617 | 2615 |
| |
2618 | 2616 |
| |
2619 |
| - | |
2620 |
| - | |
2621 | 2617 |
| |
2622 | 2618 |
| |
2623 | 2619 |
| |
| |||
2626 | 2622 |
| |
2627 | 2623 |
| |
2628 | 2624 |
| |
2629 |
| - | |
2630 |
| - | |
2631 | 2625 |
| |
2632 | 2626 |
| |
2633 |
| - | |
2634 |
| - | |
2635 | 2627 |
| |
2636 | 2628 |
| |
2637 | 2629 |
| |
| |||
2711 | 2703 |
| |
2712 | 2704 |
| |
2713 | 2705 |
| |
2714 |
| - | |
2715 |
| - | |
2716 | 2706 |
| |
2717 | 2707 |
| |
2718 | 2708 |
| |
| |||
2729 | 2719 |
| |
2730 | 2720 |
| |
2731 | 2721 |
| |
2732 |
| - | |
2733 |
| - | |
2734 | 2722 |
| |
2735 | 2723 |
| |
2736 | 2724 |
| |
| |||
2890 | 2878 |
| |
2891 | 2879 |
| |
2892 | 2880 |
| |
2893 |
| - | |
2894 |
| - | |
2895 | 2881 |
| |
2896 | 2882 |
| |
2897 |
| - | |
2898 |
| - | |
2899 | 2883 |
| |
2900 | 2884 |
| |
2901 | 2885 |
| |
|
Lines changed: 0 additions & 12 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
95 | 95 |
| |
96 | 96 |
| |
97 | 97 |
| |
98 |
| - | |
99 |
| - | |
100 | 98 |
| |
101 | 99 |
| |
102 |
| - | |
103 |
| - | |
104 | 100 |
| |
105 | 101 |
| |
106 | 102 |
| |
107 | 103 |
| |
108 | 104 |
| |
109 | 105 |
| |
110 |
| - | |
111 |
| - | |
112 | 106 |
| |
113 | 107 |
| |
114 |
| - | |
115 |
| - | |
116 | 108 |
| |
117 | 109 |
| |
118 | 110 |
| |
119 | 111 |
| |
120 | 112 |
| |
121 | 113 |
| |
122 |
| - | |
123 |
| - | |
124 | 114 |
| |
125 | 115 |
| |
126 | 116 |
| |
| |||
139 | 129 |
| |
140 | 130 |
| |
141 | 131 |
| |
142 |
| - | |
143 |
| - | |
144 | 132 |
| |
145 | 133 |
| |
146 | 134 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1642 | 1642 |
| |
1643 | 1643 |
| |
1644 | 1644 |
| |
1645 |
| - | |
1646 |
| - | |
1647 | 1645 |
| |
1648 | 1646 |
| |
1649 |
| - | |
1650 |
| - | |
1651 | 1647 |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3476 | 3476 |
| |
3477 | 3477 |
| |
3478 | 3478 |
| |
3479 |
| - | |
3480 |
| - | |
3481 | 3479 |
| |
3482 | 3480 |
| |
3483 |
| - | |
3484 |
| - | |
3485 | 3481 |
| |
3486 | 3482 |
| |
3487 | 3483 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2243 | 2243 |
| |
2244 | 2244 |
| |
2245 | 2245 |
| |
2246 |
| - | |
2247 |
| - | |
2248 | 2246 |
| |
2249 | 2247 |
| |
2250 |
| - | |
2251 |
| - | |
2252 | 2248 |
| |
2253 | 2249 |
| |
2254 | 2250 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
638 | 638 |
| |
639 | 639 |
| |
640 | 640 |
| |
641 |
| - | |
642 |
| - | |
643 | 641 |
| |
644 | 642 |
| |
645 | 643 |
| |
| |||
683 | 681 |
| |
684 | 682 |
| |
685 | 683 |
| |
686 |
| - | |
687 |
| - | |
688 | 684 |
|
Lines changed: 0 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2970 | 2970 |
| |
2971 | 2971 |
| |
2972 | 2972 |
| |
2973 |
| - | |
2974 |
| - | |
2975 | 2973 |
| |
2976 | 2974 |
| |
2977 | 2975 |
| |
| |||
2993 | 2991 |
| |
2994 | 2992 |
| |
2995 | 2993 |
| |
2996 |
| - | |
2997 |
| - | |
2998 | 2994 |
| |
2999 | 2995 |
| |
3000 | 2996 |
| |
| |||
3004 | 3000 |
| |
3005 | 3001 |
| |
3006 | 3002 |
| |
3007 |
| - | |
3008 |
| - | |
3009 | 3003 |
| |
3010 | 3004 |
| |
3011 | 3005 |
| |
| |||
3017 | 3011 |
| |
3018 | 3012 |
| |
3019 | 3013 |
| |
3020 |
| - | |
3021 |
| - | |
3022 | 3014 |
| |
3023 | 3015 |
| |
3024 | 3016 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
363 | 363 |
| |
364 | 364 |
| |
365 | 365 |
| |
366 |
| - | |
367 |
| - | |
368 | 366 |
| |
369 | 367 |
| |
370 | 368 |
| |
| |||
452 | 450 |
| |
453 | 451 |
| |
454 | 452 |
| |
455 |
| - | |
456 |
| - | |
457 | 453 |
| |
458 | 454 |
| |
459 | 455 |
| |
|
Lines changed: 0 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
152 | 152 |
| |
153 | 153 |
| |
154 | 154 |
| |
155 |
| - | |
156 | 155 |
| |
157 | 156 |
| |
158 | 157 |
| |
| |||
173 | 172 |
| |
174 | 173 |
| |
175 | 174 |
| |
176 |
| - | |
177 |
| - | |
178 | 175 |
| |
179 | 176 |
| |
180 | 177 |
| |
|
Lines changed: 6 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
80 | 80 |
| |
81 | 81 |
| |
82 | 82 |
| |
| 83 | + | |
| 84 | + | |
83 | 85 |
| |
84 | 86 |
| |
85 | 87 |
| |
86 | 88 |
| |
| 89 | + | |
| 90 | + | |
87 | 91 |
| |
88 | 92 |
| |
89 | 93 |
| |
| |||
102 | 106 |
| |
103 | 107 |
| |
104 | 108 |
| |
| 109 | + | |
| 110 | + | |
105 | 111 |
| |
106 | 112 |
| |
107 | 113 |
| |
|
0 commit comments
Comments
(0)