| От | Fabien COELHO |
|---|---|
| Тема | Re: fix psql \conninfo & \connect when using hostaddr |
| Дата | |
| Msg-id | alpine.DEB.2.21.1811192036380.4640@lancre обсуждение исходный текст |
| Ответ на | Re: fix psql \conninfo & \connect when using hostaddr (Alvaro Herrera <alvherre@2ndquadrant.com>) |
| Список | pgsql-hackers |
Hello Alvaro, >> - libpq internal function getHostaddr get a length, >> and I added an assert about it. >> - added a few braces on if/if/else/if/else/else >> - added an UNKNOWN_HOST macro to hide "???" >> - moved host_addr[] declaration earlier to avoid some braces > > You forgot to free(conn->connip) during freePGconn(). Argh, indeed:-( > I found the UNKNOWN_HOST business quite dubious -- not only in your > patch but also in the existing coding. So did I:-) I only kept it because it was already done like that. > I changed the getHostname API so that instead of returning "???" it sets > the output buffer to the empty string. I hesitated to do exactly that. I'm fine with that. Thanks for the push. -- Fabien.
В списке pgsql-hackers по дате отправления:
Сайт использует файлы cookie для корректной работы и повышения удобства. Нажимая кнопку «Принять» или продолжая пользоваться сайтом, вы соглашаетесь на их использование в соответствии с Политикой в отношении обработки cookie ООО «ППГ», в том числе на передачу данных из файлов cookie сторонним статистическим и рекламным службам. Вы можете управлять настройками cookie через параметры вашего браузера