diff options
author | Christian Grothoff <christian@grothoff.org> | 2010-09-15 07:36:41 +0000 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2010-09-15 07:36:41 +0000 |
commit | bb9de15b30d7f49dc661d2b306c192b4b50e2bce (patch) | |
tree | 5729d3a2bf58f3576074ae0ea7d82b8d8efb4ebe /AUTHORS | |
parent | 02e6d679ac3c7e712f9def8b0c4322bbb0e82d92 (diff) | |
download | libmicrohttpd-bb9de15b30d7f49dc661d2b306c192b4b50e2bce.tar.gz libmicrohttpd-bb9de15b30d7f49dc661d2b306c192b4b50e2bce.zip |
Re: [libmicrohttpd] Cleanup callback isn't called
From:
Erik Slagter <erik@slagter.name>
To:
Christian Grothoff <grothoff@net.in.tum.de>
CC:
libmicrohttpd@gnu.org
Date:
Yesterday 17:31:46
Spam Status: Spamassassin 0% probability of being spam.
Full report:
Probability=No, score=-2.6 required=7.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VERIFIED autolearn=ham version=3.2.5-tuminfo_1
> >> I think I found & fixed the issue in SVN 12778. Please try SVN HEAD and
> > > report if it fixes the problem.
> > Okay, will do so. But first I'll be on vacation for two weeks.
> Sure.
The issue indeed has been solved. It's quite a pity the distribution's
packages lag behind by ages, so again I'll have to make my own packages
now.
I've run into another problem and also I think I have found the cause.
When a connection is closed by the peer, my application tends to
segfault.
GDB says:
Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7ffff6a6c710 (LWP 27005)]
0x00007ffff77a4db0 in pthread_mutex_lock () from /lib64/libpthread.so.0
Missing separate debuginfos, use: debuginfo-install glibc-2.11.2-1.x86_64 libgcc-4.4.4-10.fc12.x86_64 libid3tag-0.15.1b-9.fc12.x86_64 libstdc++-4.4.4-10.fc12.x86_64 zlib-1.2.3-23.fc12.x86_64
(gdb) where
#0 0x00007ffff77a4db0 in pthread_mutex_lock () from /lib64/libpthread.so.0
#1 0x00007ffff7bd97ce in MHD_destroy_response (response=0x7ffff7ed5010) at response.c:341
#2 0x00007ffff7bd760f in MHD_cleanup_connections (daemon=0x62eb60) at daemon.c:964
#3 0x00007ffff7bd8fbc in MHD_select_thread (cls=0x62eb60) at daemon.c:1205
#4 0x00007ffff77a2a3a in start_thread () from /lib64/libpthread.so.0
#5 0x00007ffff6d6077d in clone () from /lib64/libc.so.6
#6 0x0000000000000000 in ?? ()
It looks like pos->response is being "destroyed" twice. If I remove one of the instances, the segfault is gone.
Index: src/daemon/daemon.c
===================================================================
--- src/daemon/daemon.c (revision 12985)
+++ src/daemon/daemon.c (working copy)
@@ -952,7 +952,7 @@
abort();
}
}
- MHD_destroy_response (pos->response);
+ // MHD_destroy_response (pos->response);
MHD_pool_destroy (pos->pool);
#if HTTPS_SUPPORT
if (pos->tls_session != NULL)
Diffstat (limited to 'AUTHORS')
-rw-r--r-- | AUTHORS | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -26,6 +26,7 @@ John Muth <muth@parascale.com> | |||
26 | Geoffrey McRae <geoff@spacevs.com> | 26 | Geoffrey McRae <geoff@spacevs.com> |
27 | Piotr Grzybowski <narsil.pl@gmail.com> | 27 | Piotr Grzybowski <narsil.pl@gmail.com> |
28 | Gerrit Telkamp <g.telkamp@domologic.de> | 28 | Gerrit Telkamp <g.telkamp@domologic.de> |
29 | Erik Slagter <erik@slagter.name> | ||
29 | 30 | ||
30 | Documentation contributions also came from: | 31 | Documentation contributions also came from: |
31 | Marco Maggi <marco.maggi-ipsu@poste.it> | 32 | Marco Maggi <marco.maggi-ipsu@poste.it> |