- Notifications
You must be signed in to change notification settings - Fork5
Commitcc0cac4
committed
Fix oversight in original coding of inline_function(): since
check_sql_fn_retval allows binary-compatibility cases, the expressionextracted from an inline-able SQL function might have a type that is onlybinary-compatible with the declared function result type. To avoid possiblychanging the semantics of the expression, we should insert a RelabelType nodein such cases. This has only been shown to have bad consequences in recent8.1 and up releases, but I suspect there may be failure cases in the olderbranches too, so patch it all the way back. Per bug #3116 from Greg Mullane.Along the way, fix an omission in eval_const_expressions_mutator: it failedto copy the relabelformat field when processing a RelabelType. No knownobservable failures from this, but it definitely isn't intended behavior.1 parent46573e6 commitcc0cac4
1 file changed
+19
-2
lines changedLines changed: 19 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
1976 | 1976 |
| |
1977 | 1977 |
| |
1978 | 1978 |
| |
| 1979 | + | |
1979 | 1980 |
| |
1980 | 1981 |
| |
1981 | 1982 |
| |
| |||
2929 | 2930 |
| |
2930 | 2931 |
| |
2931 | 2932 |
| |
2932 |
| - | |
| 2933 | + | |
| 2934 | + | |
2933 | 2935 |
| |
2934 | 2936 |
| |
2935 | 2937 |
| |
| |||
3014 | 3016 |
| |
3015 | 3017 |
| |
3016 | 3018 |
| |
| 3019 | + | |
| 3020 | + | |
| 3021 | + | |
| 3022 | + | |
| 3023 | + | |
| 3024 | + | |
| 3025 | + | |
| 3026 | + | |
| 3027 | + | |
| 3028 | + | |
| 3029 | + | |
| 3030 | + | |
| 3031 | + | |
| 3032 | + | |
| 3033 | + | |
3017 | 3034 |
| |
3018 | 3035 |
| |
3019 | 3036 |
| |
|
0 commit comments
Comments
(0)