procyberian/emacs
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove dangerous HTML edit from admin.el

Browse source 
* admin/admin.el (manual-html-fix-index-2): Avoid lax matches with
"<ul>" which could mistakenly edit unrelated parts of HTML.
(Bug#72761)
This commit is contained in:
Eli Zaretskii 2024-08-25 15:46:05 +03:00
parent 52829bcee6
commit 969498c25d
1 changed files with 1 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
admin/admin.el
View file

@ -613,9 +613,7 @@ style=\"text-align:left\">")
;; item is not there anymore. So for HTML manuals produced by ;; item is not there anymore. So for HTML manuals produced by
;; those newer versions of Texinfo we punt and leave the menu in ;; those newer versions of Texinfo we punt and leave the menu in
;; its original form. ;; its original form.
(when (or (search-forward "<ul class=\"menu\">" nil t) (when (or (search-forward "<ul class=\"menu\">" nil t))
;; FIXME? The following search seems dangerously lax.
(search-forward "<ul>" nil t))
;; Convert the list that Makeinfo made into a table. ;; Convert the list that Makeinfo made into a table.
(replace-match "<table style=\"float:left\" width=\"100%\">") (replace-match "<table style=\"float:left\" width=\"100%\">")
(forward-line 1) (forward-line 1)