On Sun, Oct 26, 2025 at 08:29:43PM -0400, Thomas Dickey wrote: ... > ...the same place. MakeFormAction could return a NULL, so... just adding > a null-pointer check for pfi before making that assignment "should" work. > Testing that change, I don't see an addtional problem.
See attached. -- Thomas E. Dickey <[email protected]> https://invisible-island.net
REV:1.259 LYMainLoop.c 2025/10/27 08:00:28 tom
Add a null pointer check to MakeFormAction (Debian #1119084).
I overlooked this one in my fixes for Debian #603648.
--- LYMainLoop.c 2025/01/08 00:06:03 1.258
+++ LYMainLoop.c 2025/10/27 08:00:28 1.259
@@ -1,5 +1,5 @@
/*
- * $LynxId: LYMainLoop.c,v 1.258 2025/01/07 21:52:11 tom Exp $
+ * $LynxId: LYMainLoop.c,v 1.258 2025/01/08 00:06:03 tom Exp $
*/
#include <HTUtils.h>
#include <HTAccess.h>
@@ -1546,7 +1546,13 @@
if (result->submit_action == NULL) {
PerFormInfo *pfi = HText_PerFormInfo(result->number);
- *result = pfi->data;
+ if (pfi == NULL) {
+ CTRACE((tfp,
+ "MakeFormAction: form %d not in HTMainText's list!\n",
+ result->number));
+ } else {
+ *result = pfi->data;
+ }
}
result->type = F_SUBMIT_TYPE;
} else {
signature.asc
Description: PGP signature
