[groff] 01/01: "grops -c a" dumps core

[Carsten Kunze]

carstenkunze pushed a commit to branch master
in repository groff.

commit bc85af4016824b6bf97f859e16722486a548ab03
Author: Rich Burridge <richburridge>
Date:   Fri Apr 28 21:48:21 2017 +0200

    "grops -c a" dumps core
    
    errprint() uses %1 etc. instead of %s
    
    Fixes https://savannah.gnu.org/bugs/?42190
---
 ChangeLog                | 10 ++++++++++
 src/devices/grops/ps.cpp |  2 +-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index b098b0a..419cbb0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2017-04-28  Rich Burridge
+
+       "grops -c a" dumps core
+
+       * src/devices/grops/ps.cpp: conversion specifier %s changed
+       to %1 (errprint() uses special syntax different from
+       printf(3))
+
+       https://savannah.gnu.org/bugs/?42190
+
 2017-04-28  Bjarni Ingi Gislason  <address@hidden>
 
        Macro "itc" is needed to make escape "\c" useful
diff --git a/src/devices/grops/ps.cpp b/src/devices/grops/ps.cpp
index 86b2b9d..5546ed2 100644
--- a/src/devices/grops/ps.cpp
+++ b/src/devices/grops/ps.cpp
@@ -1802,7 +1802,7 @@ int main(int argc, char **argv)
       break;
     case 'c':
       if (sscanf(optarg, "%d", &ncopies) != 1 || ncopies <= 0) {
-       error("bad number of copies `%s'", optarg);
+       error("bad number of copies `%1'", optarg);
        ncopies = 1;
       }
       break;