| From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
|---|---|
| To: | Ali Akbar <the(dot)apaan(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Abhijit Menon-Sen <ams(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [REVIEW] Re: Fix xpath() to return namespace definitions |
| Date: | 2014-11-04 15:13:37 |
| Message-ID: | 5458ED21.7050406@gmx.net |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 7/11/14 4:36 AM, Ali Akbar wrote:
> But i found some bug in libxml2's code, because we call xmlCopyNode with
> 1 as the second parameter, internally xmlCopyNode will call
> xmlStaticCopyNode recursively via xmlStaticCopyNodeList. And
> xmlStaticCopyNodeList doesn't check the return of xmlStaticCopyNode
> whether it's NULL. So if someday the recursion returns NULL (maybe
> because of memory exhaustion), it will SEGFAULT.
>
> Knowing this but in libxml2 code, is this patch is still acceptable?
This problem was fixed in libxml2 2.9.2 (see
https://bugzilla.gnome.org/show_bug.cgi?id=708681)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2014-11-04 15:14:40 | Re: [REVIEW] Re: Fix xpath() to return namespace definitions |
| Previous Message | Alexey Vasiliev | 2014-11-04 14:50:32 | Re[2]: [HACKERS] Patch: add recovery_timeout option to control timeout of restore_command nonzero status code |