Bug#1012020: trustedqsl: segfault when trying to renew station certificate
This is a bug introduced in TQSL 2.6.
Patch to correct this attached. This will go out as part of TQSL 2.6.4.
73,
-Rick
Package: trustedqsl
Version: 2.6.2-1
Severity: normal
Hi, creating this for visibility. Since I'm experiencing the issue, I
will try to resolve it. Also (not related to this bug), I have an
update to upstream 2.6.3 ready to upload. I am planning to wait until the
auto-openssl transition completes before uploading it.
When starting tqsl, I am prompted to renew my station certificate. If I
select "yes," the application encounters the following segmentation
fault. I'm not sure if this is an upstream bug or due to our packaging.
It appears that the code on apps/certtree.cpp line 215 fails to return a
valid tQSL_Cert. (Perhaps only in some circumstances?)
Thread 1 "tqsl" received signal SIGSEGV, Segmentation fault.
0x00005555556d023d in CertTree::SelectCert (this=0x55555682c7f0, cert=<optimized out>) at ./apps/certtree.cpp:217
217 ./apps/certtree.cpp: No such file or directory.
(gdb) bt
#0 0x00005555556d023d in CertTree::SelectCert(void*) (this=0x55555682c7f0, cert=<optimized out>) at ./apps/certtree.cpp:217
#1 0x00005555556641a5 in MyFrame::OnExpiredCertFound(wxCommandEvent&) (this=0x5555562bd8a0, event=<optimized out>) at ./apps/tqsl.cpp:3731
#2 0x00007ffff77ee15e in wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&) ()
at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#3 0x00007ffff77ee253 in wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*) () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#4 0x00007ffff77ee58c in wxEvtHandler::TryHereOnly(wxEvent&) () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#5 0x00007ffff77ee61b in wxEvtHandler::ProcessEventLocally(wxEvent&) () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#6 0x00007ffff77ee6f1 in wxEvtHandler::ProcessEvent(wxEvent&) () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#7 0x00007ffff77ef1e5 in wxEvtHandler::ProcessPendingEvents() () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#8 0x00007ffff7699f27 in wxAppConsoleBase::ProcessPendingEvents() () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#9 0x00007ffff7b428da in wxGUIEventLoop::YieldFor(long) () at /lib/x86_64-linux-gnu/libwx_gtk3u_core-3.0.so.0
#10 0x00007ffff769a8ed in wxAppConsoleBase::Yield(bool) () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#11 0x000055555562a395 in MyFrame::DoUpdateCheck(bool, bool) (this=0x5555562bd8a0, silent=<optimized out>, noGUI=<optimized out>)
at /usr/include/wx-3.0/wx/app.h:439
#12 0x00005555556616df in MyFrame::FirstTime() (this=0x5555562bd8a0) at ./apps/tqsl.cpp:6093
#13 0x0000555555671895 in QSLApp::GUIinit(bool, bool) (this=0x555555804c20, checkUpdates=<optimized out>, quiet=<optimized out>)
at ./apps/tqsl.cpp:5243
#14 0x0000555555672f9a in QSLApp::OnInit() (this=0x555555804c20) at ./apps/tqsl.cpp:5460
#15 0x00007ffff7714a72 in wxEntry(int&, wchar_t**) () at /lib/x86_64-linux-gnu/libwx_baseu-3.0.so.0
#16 0x000055555561cd1e in main(int, char**) (argc=<optimized out>, argv=<optimized out>) at ./apps/tqsl.cpp:234
-- System Information:
Debian Release: bookworm/sid
APT prefers unstable
APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)
Kernel: Linux 5.17.0-3-amd64 (SMP w/8 CPU threads; PREEMPT)
Kernel taint flags: TAINT_WARN
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled
Versions of packages trustedqsl depends on:
ii libc6 2.33-7
ii libcurl4 7.83.1-1+b1
ii libexpat1 2.4.8-1
ii libgcc-s1 12.1.0-2
ii liblmdb0 0.9.24-1
ii libssl3 3.0.3-5
ii libstdc++6 12.1.0-2
ii libwxbase3.0-0v5 3.0.5.1+dfsg-4
ii libwxgtk3.0-gtk3-0v5 3.0.5.1+dfsg-4
ii zlib1g 1:1.2.11.dfsg-4
trustedqsl recommends no packages.
trustedqsl suggests no packages.
-- no debconf information
--
Rick Murphy, D.Sc., CISSP-ISSAP, K1MU/4, Annandale VA USA
diff --git a/apps/ChangeLog.txt b/apps/ChangeLog.txt
index e43836f..45fd660 100644
--- a/apps/ChangeLog.txt
+++ b/apps/ChangeLog.txt
@@ -1,5 +1,11 @@
TQSL changes
============
+2.6.4
+----------
+Correct a crash when TQSL detects a callsign certificate that's about to
+expire during startup and the user selects "Yes" to renew that callsign
+certificate.
+
2.6.3
----------
Handle the case where there's both a valid and an invalid callsign certificate
diff --git a/apps/certtree.cpp b/apps/certtree.cpp
index 272e642..55f83f5 100644
--- a/apps/certtree.cpp
+++ b/apps/certtree.cpp
@@ -202,6 +202,7 @@ CertTree::SelectCert(tQSL_Cert cert) {
// Iterate the tree, looking for a matching certificate
wxTreeItemId root = GetRootItem();
wxTreeItemIdValue issCookie;
+ wxTreeItemIdValue listsCookie;
wxTreeItemIdValue certCookie;
wxTreeItemId top;
if (_nissuers > 1) {
@@ -210,16 +211,20 @@ CertTree::SelectCert(tQSL_Cert cert) {
top = root;
}
while (top.IsOk()) {
- wxTreeItemId item = GetFirstChild(top, certCookie);
+ wxTreeItemId item = GetFirstChild(top, listsCookie);
while (item.IsOk()) {
- tQSL_Cert cert = GetItemData(item)->getCert();
- long s;
- tqsl_getCertificateSerial(cert, &s);
- if (s == serial) { // found it
- SelectItem(item);
- return;
+ wxTreeItemId subitem = GetFirstChild(item, certCookie);
+ while (subitem.IsOk()) {
+ tQSL_Cert cert = GetItemData(subitem)->getCert();
+ long s;
+ tqsl_getCertificateSerial(cert, &s);
+ if (s == serial) { // found it
+ SelectItem(subitem);
+ return;
+ }
+ subitem = GetNextChild(item, certCookie);
}
- item = GetNextChild(top, certCookie);
+ item = GetNextChild(top, listsCookie);
}
if (_nissuers > 1) {
top = GetNextChild(root, issCookie);
Reply to: