aboutsummaryrefslogtreecommitdiffstats
path: root/debian/patches/908_forms.patch
diff options
context:
space:
mode:
authorTatsuya Kinoshita <tats@debian.org>2016-11-21 14:06:13 +0000
committerTatsuya Kinoshita <tats@debian.org>2016-11-21 14:06:13 +0000
commit6370b839fbe60331342e850731ff9d10328de1af (patch)
treed962eaa9bb4a7346db3fe0e866eaf998fa1174f6 /debian/patches/908_forms.patch
parentNew patch 907_iso2022.patch to fix array index [CVE-2016-9433] (diff)
downloadw3m-6370b839fbe60331342e850731ff9d10328de1af.tar.gz
w3m-6370b839fbe60331342e850731ff9d10328de1af.zip
New patch 908_forms.patch to fix null deref [CVE-2016-9434]
Diffstat (limited to 'debian/patches/908_forms.patch')
-rw-r--r--debian/patches/908_forms.patch28
1 files changed, 28 insertions, 0 deletions
diff --git a/debian/patches/908_forms.patch b/debian/patches/908_forms.patch
new file mode 100644
index 0000000..d38c665
--- /dev/null
+++ b/debian/patches/908_forms.patch
@@ -0,0 +1,28 @@
+Subject: Prevent segfault with incorrect form_int fid
+Author: Tatsuya Kinoshita <tats@debian.org>
+Bug-Debian: https://github.com/tats/w3m/issues/15 [CVE-2016-9434]
+Origin: https://anonscm.debian.org/cgit/collab-maint/w3m.git/commit/?id=3d4eeda9ec0cb91e23bab7dc260d4c515119eb4b
+
+diff --git a/file.c b/file.c
+index 4056393..b30aa6b 100644
+--- a/file.c
++++ b/file.c
+@@ -4172,7 +4172,7 @@ process_form_int(struct parsed_tag *tag, int fid)
+ forms = New_N(FormList *, forms_size);
+ form_stack = NewAtom_N(int, forms_size);
+ }
+- else if (forms_size <= form_max) {
++ if (forms_size <= form_max) {
+ forms_size += form_max;
+ forms = New_Reuse(FormList *, forms, forms_size);
+ form_stack = New_Reuse(int, form_stack, forms_size);
+@@ -6086,7 +6086,8 @@ HTMLlineproc2body(Buffer *buf, Str (*feed) (), int llimit)
+ fclose(debug);
+ #endif
+ for (form_id = 1; form_id <= form_max; form_id++)
+- forms[form_id]->next = forms[form_id - 1];
++ if (forms[form_id])
++ forms[form_id]->next = forms[form_id - 1];
+ buf->formlist = (form_max >= 0) ? forms[form_max] : NULL;
+ if (n_textarea)
+ addMultirowsForm(buf, buf->formitem);