Re: [hail patch 1/1] fix hstor crashes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/07/2010 08:21 PM, Pete Zaitcev wrote:
This patch addresses two bugs:

- No parameters to search worked (because character '=' was missing,
   e.g.  delim/ instead of delim=/), and also there were weird truncations
   because prefix can easily be longer than 32 characters.

- If prefix is empty, xmlNodeListGetString returns NULL instead of "",
   thus segfault in strdup. This is not easy to accomplish with Boto,
   but our own httpstor can do it.

Signed-off-by: Pete Zaitcev<zaitcev@xxxxxxxxxx>

---
  lib/hstor.c |    6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)

This patch was actually tested on httpstor, I've not switched to hstor
yet, but it seems to apply 100%. There's no sense in fixing httpstor,
I'm going to delete it from tabled soon.

applied

Check out 'libhail-merge' branch of tabled.git, it should have already fixed up the issues and the delete.


--
To unsubscribe from this list: send the line "unsubscribe hail-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Fedora Clound]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux