dtprintinfo: Fix SIGSEGV in dtprintinfo (passing NULL as first argument to catgets).
authorJon Trulson <jon@radscan.com>
Mon, 6 Aug 2012 23:12:38 +0000 (17:12 -0600)
committerJon Trulson <jon@radscan.com>
Mon, 6 Aug 2012 23:12:38 +0000 (17:12 -0600)
commit606df517f0aecafd32a45fe981321d5aea35f4c8
tree0aafd2cc532e3b61fec8612e3547997343d9f76f
parent1a03c2594780e24ee92a0c2ee49d22b16c3fda0c
dtprintinfo: Fix SIGSEGV in dtprintinfo (passing NULL as first argument to catgets).

Patch from Douglas Mencken <dougmencken@gmail.com>:

The issue is that MESSAGE tries to invoke catgets with NULL first
parameter, which is dereferenced inside catgets (Std C Library
function) without checking, from catgets.c source:

if (catalog->name_ptr[idx + 0] == (u_int32_t) set
          && catalog->name_ptr[idx + 1] == (u_int32_t) message)

On the other hand, there's a special value: -1 (cast to nl_catd),
which must be used instead of 0 (NULL) in the case when we are unable
to provide real catalog_desc, from catgets.c source:

  /* Be generous if catalog which failed to be open is used.  */
  if (catalog_desc == (nl_catd) -1 || ++set <= 0 || message < 0)
    return (char *) string;
cde/programs/dtprintinfo/DtPrintinfo.C