]> git.eshelyaron.com Git - emacs.git/commitdiff
Remove dangerous HTML edit from admin.el
authorEli Zaretskii <eliz@gnu.org>
Sun, 25 Aug 2024 12:46:05 +0000 (15:46 +0300)
committerEshel Yaron <me@eshelyaron.com>
Wed, 4 Sep 2024 07:51:32 +0000 (09:51 +0200)
* admin/admin.el (manual-html-fix-index-2): Avoid lax matches with
"<ul>" which could mistakenly edit unrelated parts of HTML.
(Bug#72761)

(cherry picked from commit 969498c25d0112dbdc3aa2ef75dc63681101203e)

admin/admin.el

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