Bug 6977 - Bad name for syslog in gatekeeper
: Bad name for syslog in gatekeeper
Status: RESOLVED FIXED
: GRAM
gt2 Gatekeeper/Jobmanager
: unspecified
: PC All
: P3 normal
: 5.0.1
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2010-03-08 22:21 by
Modified: 2010-03-09 09:07 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2010-03-08 22:21:57
Hi,

EGEE reported a bug to me, and I'd like to pass it on to you. It's a trivial
fix--just a single character. It exists in Globus 4.0 and Globus 5.0, but I'm
happy if you just fix it in 5.0. In fact, it's such an easy fix that maybe you
can get it in before the 5.0.1 release. :)

You can see the full bug here:
https://savannah.cern.ch/bugs/?49324

I'll summarize it for you though. 

Look in globus_gatekeeper.c, around line 2491, and you'll see a line like this:

        openlog("GRAM gatekeeper", LOG_PID, LOG_DAEMON);

It's not obvious from the man page for openlog, but the syslog protocol doesn't
allow space characters. So you should change it to something like:

        openlog("GRAM-gatekeeper", LOG_PID, LOG_DAEMON);

Of course, if you prefer a different name, that's cool too. Just something that
matches the syslog protocol. 

This is causing problems at CERN for an alternative implementation of syslog,
so it's not just a theoretical problem. Of course, I'll have to patch it for
them because they are using an older version of Globus, but it would be nice if
you fixed it in your implementation moving forward.

Thanks!
-alain
------- Comment #1 From 2010-03-09 09:07:49 -------
Committed to 5.0 branch and trunk.