<p> Attention is currently required from: Sean Bright, N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18829">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18829?tab=comments">Patch Set #18:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">Patchset 18:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Progress, but broken.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Took Joshua's advice, don't allow unload_module, asterisk crashed with double free errors if a phonecall hungup after module was unloaded.</p><p style="white-space: pre-wrap; word-wrap: break-word;">However (but unrelated to these changes) after every thread cleanup (when caller hangs-up) I'm getting </p><p style="white-space: pre-wrap; word-wrap: break-word;">[Aug  3 09:34:55] DEBUG[824496]: res_hiredis.c:323 hiredis_thread_cleanup: socket_fd[32] released<br>double free or corruption (out)<br>Aborted (core dumped)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thread 1 (Thread 0x7feeb0382640 (LWP 824496) (Exiting)):<br>#0  __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:49<br>#1  0x00007feeb279b546 in __GI_abort () at abort.c:79<br>#2  0x00007feeb27f2ed8 in __libc_message (action=action@entry=do_abort, fmt=fmt@entry=0x7feeb2910c2f "%sn") at ../sysdeps/posix/libc_fatal.c:155<br>#3  0x00007feeb27fa93a in malloc_printerr (str=str@entry=0x7feeb2913140 "double free or corruption (out)") at malloc.c:5628<br>#4  0x00007feeb27fc1c0 in _int_free (av=0x7feeb2947b80 <main_arena>, p=0x5631795ad780, have_lock=<optimized out>, have_lock@entry=0) at malloc.c:4547<br>#5  0x00007feeb27ff9d4 in __GI___libc_free (mem=<optimized out>) at malloc.c:3309<br>#6  0x00007fee90656d65 in hiredis_thread_cleanup (data=0x5631795ad790) at res_hiredis.c:325<br>#7  0x00007feeb31bef11 in __nptl_deallocate_tsd () at pthread_create.c:311<br>#8  0x00007feeb31bfda1 in __nptl_deallocate_tsd () at pthread_create.c:266<br>#9  start_thread (arg=0x7feeb0382640) at pthread_create.c:492<br>#10 0x00007feeb2873baf in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95</p><p style="white-space: pre-wrap; word-wrap: break-word;">What I don't know is how on thread creation how to properly allocate enough threadstorage the size of redisContext, I must be clobbering something when I memcpy the newly allocated memory from redisConnectWithTimeout to threadstorage.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Any guidance would be appreciated.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Alec<br></p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18829">change 18829</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/18829"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ifdf4d33d2a2a5fd044fac13b201c7437de34ba6a </div>
<div style="display:none"> Gerrit-Change-Number: 18829 </div>
<div style="display:none"> Gerrit-PatchSet: 18 </div>
<div style="display:none"> Gerrit-Owner: Alec Davis <alec@bdt.co.nz> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 02 Aug 2022 21:46:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>