Re: UT_strdup Patch

From: Hubert Figuiere (hfiguiere@teaser.fr)
Date: Thu May 23 2002 - 17:28:28 EDT

  • Next message: Hubert Figuiere: "Re: STABLE dies on launch"

    According to Christian Biesinger <cbiesinger@web.de>:
    > On Thu, May 23, 2002 at 01:47:52PM -0700, Jeshua Lacock wrote:
    > > char * UT_strdup(const char * szSource)
    > > {
    > > + if (szSource == NULL)
    > > + return(NULL);
    > > +
    >
    > This looks like it just hides the real problem, being that UT_strdup is
    > passed null.
    >
    > Can we fix the caller instead of penalizing everybody who calls UT_strdup?

    If it is the LANG env that is NULL, then this has been fixed and
    should be in 1.0.2.

    I'll let that alone. HEAD does have a better check with the new assert
    code.


    Hub


    This archive was generated by hypermail 2.1.4 : Thu May 23 2002 - 17:32:51 EDT