=================================================================== RCS file: /cvs/mandoc/mdoc_validate.c,v retrieving revision 1.200 retrieving revision 1.201 diff -u -p -r1.200 -r1.201 --- mandoc/mdoc_validate.c 2014/01/06 22:39:25 1.200 +++ mandoc/mdoc_validate.c 2014/01/07 09:10:45 1.201 @@ -1,4 +1,4 @@ -/* $Id: mdoc_validate.c,v 1.200 2014/01/06 22:39:25 schwarze Exp $ */ +/* $Id: mdoc_validate.c,v 1.201 2014/01/07 09:10:45 schwarze Exp $ */ /* * Copyright (c) 2008-2012 Kristaps Dzonsons * Copyright (c) 2010-2014 Ingo Schwarze @@ -2348,12 +2348,13 @@ post_bx(POST_ARGS) static int post_os(POST_ARGS) { - struct mdoc_node *n; char buf[BUFSIZ]; - int c; #ifndef OSNAME struct utsname utsname; + static char *defbuf; #endif + struct mdoc_node *n; + int c; n = mdoc->last; @@ -2376,39 +2377,31 @@ post_os(POST_ARGS) assert(c); - if ('\0' == buf[0]) { - if (mdoc->defos) { - mdoc->meta.os = mandoc_strdup(mdoc->defos); - return(1); - } + if ('\0' != *buf) { + mdoc->meta.os = mandoc_strdup(buf); + return(1); + } + + if (mdoc->defos) { + mdoc->meta.os = mandoc_strdup(mdoc->defos); + return(1); + } + #ifdef OSNAME - if (strlcat(buf, OSNAME, BUFSIZ) >= BUFSIZ) { - mdoc_nmsg(mdoc, n, MANDOCERR_MEM); - return(0); - } + mdoc->meta.os = mandoc_strdup(OSNAME); #else /*!OSNAME */ + if (NULL == defbuf) { if (-1 == uname(&utsname)) { mdoc_nmsg(mdoc, n, MANDOCERR_UNAME); - mdoc->meta.os = mandoc_strdup("UNKNOWN"); - return(post_prol(mdoc)); - } - - if (strlcat(buf, utsname.sysname, BUFSIZ) >= BUFSIZ) { - mdoc_nmsg(mdoc, n, MANDOCERR_MEM); - return(0); + defbuf = mandoc_strdup("UNKNOWN"); + } else if (-1 == asprintf(&defbuf, "%s %s", + utsname.sysname, utsname.release)) { + perror(NULL); + exit((int)MANDOCLEVEL_SYSERR); } - if (strlcat(buf, " ", BUFSIZ) >= BUFSIZ) { - mdoc_nmsg(mdoc, n, MANDOCERR_MEM); - return(0); - } - if (strlcat(buf, utsname.release, BUFSIZ) >= BUFSIZ) { - mdoc_nmsg(mdoc, n, MANDOCERR_MEM); - return(0); - } -#endif /*!OSNAME*/ } - - mdoc->meta.os = mandoc_strdup(buf); + mdoc->meta.os = mandoc_strdup(defbuf); +#endif /*!OSNAME*/ return(1); }