diff options
author | Teodor Sigaev <teodor@sigaev.ru> | 2006-08-08 15:45:24 +0000 |
---|---|---|
committer | Teodor Sigaev <teodor@sigaev.ru> | 2006-08-08 15:45:24 +0000 |
commit | 73761f3659dece410899a3876193e5cbf3a6ab9d (patch) | |
tree | dda1d832daf5ea458799dd92f47de71920b77d3c | |
parent | 804e7c5c53a937e7984b90c9fa8fcce17e952591 (diff) |
Fix bug introduced by last patch, thanks again to Mario Weilguni <mweilguni@sime.com>
-rw-r--r-- | contrib/ltree/ltree_gist.c | 30 |
1 files changed, 25 insertions, 5 deletions
diff --git a/contrib/ltree/ltree_gist.c b/contrib/ltree/ltree_gist.c index b5e25700887..c604912bc60 100644 --- a/contrib/ltree/ltree_gist.c +++ b/contrib/ltree/ltree_gist.c @@ -455,16 +455,36 @@ gist_isparent(ltree_gist * key, ltree * query) return false; } +static ltree * +copy_ltree( ltree *src ) { + ltree *dst = (ltree*)palloc(src->len); + memcpy(dst, src, src->len); + return dst; +} + static bool gist_ischild(ltree_gist * key, ltree * query) { - if (ltree_compare(query, LTG_GETLNODE(key)) < 0) - return false; + ltree *left = copy_ltree(LTG_GETLNODE(key)); + ltree *right = copy_ltree(LTG_GETRNODE(key)); + bool res = true; - if (ltree_compare(query, LTG_GETRNODE(key)) > 0) - return false; + if (left->numlevel > query->numlevel) + left->numlevel = query->numlevel; - return true; + if (ltree_compare(query, left) < 0) + res = false; + + if (right->numlevel > query->numlevel) + right->numlevel = query->numlevel; + + if (res && ltree_compare(query, right) > 0) + res = false; + + pfree(left); + pfree(right); + + return res; } static bool |