bugGNU roff - Bugs: bug #63398, [mm] mmse.tmac makes footer...

 
 

bug #63398: [mm] mmse.tmac makes footer malfunction when footnote used and page full

Submitter:  G. Branden Robinson <gbranden>
Submitted:  Tue 22 Nov 2022 04:51:21 AM UTC
   
 
Category:  Macro mm Severity:  3 - Normal
Item Group:  Incorrect behaviour Status:  Fixed
Privacy:  Public Assigned to:  gbranden
Open/Closed:  Closed Planned Release:  1.23.0
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 25 Nov 2022 03:12:33 AM UTC, comment #2: 


commit 45103c8c6273afb8cff6f2ad69522f5cb898356e
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Tue Nov 22 00:10:22 2022 -0600

    [mm]: Fix Savannah #63398.

    * contrib/mm/mse.tmac: Set `pg*footer-size` to four vees, not four basic
      units; the latter is too small by orders of magnitude on typesetting
      output devices.

    Fixes <https://savannah.gnu.org/bugs/?63398>.  Thanks to Hans Bezemer
    for the report.

commit 6661f7c48032b8a149cc361bb18e7e838ef79f15
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Mon Nov 21 23:14:21 2022 -0600

    [mmse]: Regression-test Savannah #63398.

    * contrib/mm/tests/mse_has-sufficient-footnote-space.sh: Do it.
    * contrib/mm/mm.am (mm_TESTS): Run test.

    Test fails at this commit.


G. Branden Robinson <gbranden>
Group administrator
Tue 22 Nov 2022 05:11:46 AM UTC, comment #1: 

Another condition for causing the bug is that the page with the footnote has to be otherwise full.  That could explain why possibly minimal testing never turned it up.

G. Branden Robinson <gbranden>
Group administrator
Tue 22 Nov 2022 04:51:21 AM UTC, original submission:  

Hans Bezemer reported an input stack overflow while processing the page footer when using the mmse.tmac localization file for Swedish.

(The formatter exits in a controlled way; this stack is not the execution stack employed by the OS process.  It would be an infinite loop if the `slimit` register of GNU troff were not set to a small finite value.)

https://lists.gnu.org/archive/html/groff/2022-11/msg00137.html

The problem is a missing scaling indicator in the macro file.


diff --git a/contrib/mm/mse.tmac b/contrib/mm/mse.tmac
index dedfb10bb..8184b1434 100644
--- a/contrib/mm/mse.tmac
+++ b/contrib/mm/mse.tmac
@@ -43,7 +43,7 @@ http://savannah.gnu.org/bugs/?group=groff.
 .po \n[@po]u
 .pl \n[@pl]u
 .
-.nr pg*footer-size 4\"                  1v+footer+even/odd footer+1v
+.nr pg*footer-size 4v\"                  1v+footer+even/odd footer+1v
 .\"------------------------------------------------
 .\" Dokumentnamn
 .ds LetDNAMN


Problem affects groff 1.22.4 and appears to date back 22 years, to the introduction of the file.  That's a DNAMN long time!

G. Branden Robinson <gbranden>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gbranden (Submitted the item)
  • -email is unavailable- added by gbranden (original reporter)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-11-25 gbranden StatusIn Progress Fixed
        Open/ClosedOpen Closed
    2022-11-22 gbranden Summary[mm] mmse.tmac makes footer malfunction when footnote used [mm] mmse.tmac makes footer malfunction when footnote used and page full
    2022-11-22 gbranden Planned ReleaseNone 1.23.0
    2022-11-22 gbranden Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code