forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitba06acf
Free correctly LDAPMessage returned by ldap_search_s() in auth.c
The LDAP wiki states that the search message should be freed regardlessof the return value of ldap_search_s(), but we failed to do so in onebackend code path when searching LDAP with a filter. This is notcritical in an authentication code path failing in the backend as thiscauses such the process to exit promptly, but let's be clean and freethe search message appropriately, as documented by upstream.All the other code paths failing a LDAP operation do that already, andsomebody looking at this code in the future may miss what LDAP expectswith the search message.Author: Zhihong YuDiscussion:https://postgr.es/m/CALNJ-vTf5Y+8RtzZ4GjOGE9qWVHZ8awfhnFYc_qGm8fMLUNRAg@mail.gmail.com1 parent813c884 commitba06acf
1 file changed
+3
-0
lines changedLines changed: 3 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2568 | 2568 |
| |
2569 | 2569 |
| |
2570 | 2570 |
| |
| 2571 | + | |
2571 | 2572 |
| |
2572 | 2573 |
| |
2573 | 2574 |
| |
| |||
2582 | 2583 |
| |
2583 | 2584 |
| |
2584 | 2585 |
| |
| 2586 | + | |
| 2587 | + | |
2585 | 2588 |
| |
2586 | 2589 |
| |
2587 | 2590 |
| |
|
0 commit comments
Comments
(0)