|
I was asked to run a gdb, but I don't really think it looks that helpful. I've attached it.
|
|
gdb file, looks pretty useless without a php debug symbols version
|
My bad. Type 'bt' when you get the segfault :)
That'll show you the stack trace
|
|
Yes, this looks much more helpful :)
|
|
Here is the same segfault when using the latest couchbase.so compiled from the github master branch (so, cutting edge, I'd seen some lcb changes, so I figured I'd give ya both).
|
Can you try this patch?
diff --git a/views.c b/views.c
index 6c7bc30..7d03c5a 100644
--- a/views.c
+++ b/views.c
@@ -52,12 +52,13 @@ void php_couchbase_complete_callback(lcb_http_request_t request,
return;
}
+ /** We have one extra byte in 'data' */
hti = emalloc(sizeof(*hti) + resp->v.v0.nbytes);
hti->ndata = resp->v.v0.nbytes;
- memcpy(hti->data, resp->v.v0.bytes, hti->ndata + 1);
+
if (hti->ndata) {
- hti->data[hti->ndata - 1] = '\0';
- hti->ndata--;
+ memcpy(hti->data, resp->v.v0.bytes, hti->ndata);
+ hti->data[hti->ndata] = '\0';
}
ctx->res->rc = error;
I'll make it as a gerrit changeset as well..
|
|
http://review.couchbase.org/22573
|
|
That changed fixed the segfault, yes. The value assigned to the variable from the $cb->view() call is bool(false)
|
|
Thanks for the quick work Mark! Looks like it'll be closeable after it's reviewed and in.
|
Generated at Wed May 22 22:42:03 CDT 2013 using JIRA 5.2.4#845-sha1:c9f4cc41abe72fb236945343a1f485c2c844dac9.
cbt.php
gdb-bt-git.txt