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