Tom Lane wrote: > Alvaro Herrera <[EMAIL PROTECTED]> writes: > > Tom Lane wrote: > >> Hm. But I think we'd *like* it to segfault; the idea is to make the > >> user's programming error as obvious as possible. Is it worth the > >> trouble to just zero out the pointer members of the PGresult? > > > There are only five of them; four need to be zeroed out. > > Works for me. Please commit, as I'm about to do some further work in > those files and would rather not have to merge ...
Done. They were actually four, not five. The one I mistakingly though was one was the notice processor hooks. The case Martijn was saying would be warned about by the memset approach, setting ntuples to 0, would actually be handled as a segfault, because functions like check_field_number actually follow res.noticeHooks pointer! ISTM we would just segfault at that point. -- Alvaro Herrera http://www.CommandPrompt.com/ The PostgreSQL Company - Command Prompt, Inc. ---------------------------(end of broadcast)--------------------------- TIP 6: explain analyze is your friend
