Re: Application name patch - v3

On Wed, Nov 25, 2009 at 10:01 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> ISTM restricting the name to ASCII-only is the most reasonable tradeoff.
> Of course, as a speaker of English I may be a bit biased here --- but
> doing nothing about the issue doesn't seem acceptable.

OK - something like this? Should keep non-printable/control characters
out of logs too...

static const char *
assign_application_name(const char *newval, bool doit, GucSource source)
{
        /* Only allow clean ASCII chars in the application name */
        int x;

        char *repval = guc_malloc(ERROR, strlen(newval) + 1);
        repval[0] = 0;

        for (x=0; x<strlen(newval); x++)
        {
                if (newval[x] < 32 || newval[x] > 126)
                        repval[x] = '?';
                else
                        repval[x] = newval[x];
        }

        repval[x+1] = 0;
        return repval;
}


-- 
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com