Date: Tue, 17 Jun 1997 16:27:07 -0600 (MDT) From: stephen clawson <sclawson@marker.cs.utah.edu> To: FreeBSD-gnats-submit@FreeBSD.ORG Subject: gnu/3894: manpath segfaults if it dosen't understand a line in manpath.config. Message-ID: <199706172227.QAA29888@marker.cs.utah.edu> Resent-Message-ID: <199706172230.PAA12465@hub.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 3894 >Category: gnu >Synopsis: manpath segfaults if it dosen't understand a line in manpath.config. >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-bugs >State: open >Class: sw-bug >Submitter-Id: current-users >Arrival-Date: Tue Jun 17 15:30:00 PDT 1997 >Last-Modified: >Originator: stephen clawson >Organization: // stephen clawson sclawson@cs.utah.edu // university of utah >Release: FreeBSD 2.1.7-RELEASE i386 >Environment: Standard FreeBSD 2.1.7-RELEASE manpath binary on a ppro 200. >Description: manpath segfaults if it dosen't understand a line in manpath.config because it's calling gripe_reading_mp_config without an argument. >How-To-Repeat: Put anything nonstandard in /etc/manpath.config (ie. add a `foo' at the end), or misspell one of the keywords like I did. =) >Fix: Either hardcode the config file name in gripe_reading_mp_config, or add the required argument. This seems to affect all versions of FreeBSD and should probably be fixed in _both_ copies of manpath.c (/usr/src/gnu/usr.bin/man/{manpath,man}/manpath.c). diff -u -r1.2 manpath.c --- manpath.c 1995/05/30 05:02:05 1.2 +++ manpath.c 1997/06/17 21:24:47 @@ -135,7 +135,7 @@ char *get_manpath (); if (get_dirlist ()) - gripe_reading_mp_config (); + gripe_reading_mp_config (config_file); if ((manpathlist = getenv ("MANPATH")) != NULL) /* @@ -254,7 +254,7 @@ } else { - gripe_reading_mp_config (); + gripe_reading_mp_config (config_file); } dlp++; } >Audit-Trail: >Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199706172227.QAA29888>