From 8618cc00b014dde7e7a3c20740757cab0ff0b007 Mon Sep 17 00:00:00 2001 From: Rainer Gerhards Date: Mon, 30 Jul 2007 12:19:20 +0000 Subject: moved doBinaryOption() and doGetGUID() to cfsysline.c --- cfsysline.c | 115 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ cfsysline.h | 4 ++ srUtils.c | 22 +++++++++++ srUtils.h | 1 + syslogd.c | 126 +++++------------------------------------------------------- 5 files changed, 152 insertions(+), 116 deletions(-) diff --git a/cfsysline.c b/cfsysline.c index 38ef3119..cfa8c3ee 100644 --- a/cfsysline.c +++ b/cfsysline.c @@ -27,15 +27,130 @@ #include #include #include +#include #include "rsyslog.h" +#include "syslogd.h" /* TODO: when the module interface & library design is done, this should be able to go away */ #include "cfsysline.h" +#include "srUtils.h" /* static data */ cslCmd_t *pCmdListRoot = NULL; /* The list of known configuration commands. */ cslCmd_t *pCmdListLast = NULL; +/* --------------- START functions for handling canned syntaxes --------------- */ + +/* Parse and interpret an on/off inside a config file line. This is most + * often used for boolean options, but of course it may also be used + * for other things. The passed-in pointer is updated to point to + * the first unparsed character on exit. Function emits error messages + * if the value is neither on or off. It returns 0 if the option is off, + * 1 if it is on and another value if there was an error. + * rgerhards, 2007-07-15 + */ +static int doParseOnOffOption(uchar **pp) +{ + uchar *pOptStart; + uchar szOpt[32]; + + assert(pp != NULL); + assert(*pp != NULL); + + pOptStart = *pp; + skipWhiteSpace(pp); /* skip over any whitespace */ + + if(getSubString(pp, (char*) szOpt, sizeof(szOpt) / sizeof(uchar), ' ') != 0) { + logerror("Invalid $-configline - could not extract on/off option"); + return -1; + } + + if(!strcmp((char*)szOpt, "on")) { + return 1; + } else if(!strcmp((char*)szOpt, "off")) { + return 0; + } else { + logerrorSz("Option value must be on or off, but is '%s'", (char*)pOptStart); + return -1; + } +} + + +/* extract a username and return its uid. + * rgerhards, 2007-07-17 + */ +rsRetVal doGetUID(uchar **pp, rsRetVal (*pSetHdlr)(void*, uid_t), void *pVal) +{ + struct passwd *ppwBuf; + struct passwd pwBuf; + rsRetVal iRet = RS_RET_OK; + uchar szName[256]; + char stringBuf[2048]; /* I hope this is large enough... */ + + assert(pp != NULL); + assert(*pp != NULL); + assert(pVal != NULL); + + if(getSubString(pp, (char*) szName, sizeof(szName) / sizeof(uchar), ' ') != 0) { + logerror("could not extract user name"); + iRet = RS_RET_NOT_FOUND; + goto finalize_it; + } + + getpwnam_r((char*)szName, &pwBuf, stringBuf, sizeof(stringBuf), &ppwBuf); + + if(ppwBuf == NULL) { + logerrorSz("ID for user '%s' could not be found or error", (char*)szName); + } else { + if(pSetHdlr == NULL) { + /* we should set value directly to var */ + *((uid_t*)pVal) = ppwBuf->pw_uid; + } else { + /* we set value via a set function */ + pSetHdlr(pVal, ppwBuf->pw_uid); + } + dprintf("uid %d obtained for user '%s'\n", ppwBuf->pw_uid, szName); + } + + skipWhiteSpace(pp); /* skip over any whitespace */ + +finalize_it: + return iRet; +} + + +/* Parse and process an binary cofig option. pVal must be + * a pointer to an integer which is to receive the option + * value. + * rgerhards, 2007-07-15 + */ +rsRetVal doBinaryOptionLine(uchar **pp, rsRetVal (*pSetHdlr)(void*, int), void *pVal) +{ + int iOption; + rsRetVal iRet = RS_RET_OK; + + assert(pp != NULL); + assert(*pp != NULL); + assert(pVal != NULL); + + if((iOption = doParseOnOffOption(pp)) == -1) + return RS_RET_ERR; /* nothing left to do */ + + if(pSetHdlr == NULL) { + /* we should set value directly to var */ + *((int*)pVal) = iOption; + } else { + /* we set value via a set function */ + pSetHdlr(pVal, iOption); + } + + skipWhiteSpace(pp); /* skip over any whitespace */ + return iRet; +} + + +/* --------------- END functions for handling canned syntaxes --------------- */ + /* destructor for cslCmdHdlr */ rsRetVal cslchDestruct(cslCmdHdlr_t *pThis) diff --git a/cfsysline.h b/cfsysline.h index 39083667..78b2fec2 100644 --- a/cfsysline.h +++ b/cfsysline.h @@ -65,4 +65,8 @@ rsRetVal cslchConstruct(cslCmdHdlr_t **ppThis); rsRetVal cslchSetEntry(cslCmdHdlr_t *pThis, ecslCmdHdrlType eType, rsRetVal (*pHdlr)(), void *pData); rsRetVal cslchCallHdlr(cslCmdHdlr_t *pThis, uchar **ppConfLine); +/* the next ones go away later */ +rsRetVal doBinaryOptionLine(uchar **pp, rsRetVal (*pSetHdlr)(void*, int), void *pVal); +rsRetVal doGetUID(uchar **pp, rsRetVal (*pSetHdlr)(void*, uid_t), void *pVal); + #endif /* #ifndef CFSYSLINE_H_INCLUDED */ diff --git a/srUtils.c b/srUtils.c index 6080d607..4e91bb9c 100755 --- a/srUtils.c +++ b/srUtils.c @@ -36,6 +36,7 @@ #include #include #include +#include #include "rsyslog.h" /* THIS IS A MODIFICATION FOR RSYSLOG! 2004-11-18 rgerards */ #include "liblogging-stub.h" /* THIS IS A MODIFICATION FOR RSYSLOG! 2004-11-18 rgerards */ #define TRUE 1 @@ -208,6 +209,27 @@ int execProg(uchar *program, int bWait, uchar *arg) perror("exec"); exit(1); /* not much we can do in this case */ } + + +/* skip over whitespace in a standard C string. The + * provided pointer is advanced to the first non-whitespace + * charater or the \0 byte, if there is none. It is never + * moved past the \0. + */ +void skipWhiteSpace(uchar **pp) +{ + register uchar *p; + + assert(pp != NULL); + assert(*pp != NULL); + + p = *pp; + while(*p && isspace((int) *p)) + ++p; + *pp = p; +} + + /* * vi:set ai: */ diff --git a/srUtils.h b/srUtils.h index ece63fd8..a4e70214 100755 --- a/srUtils.h +++ b/srUtils.h @@ -62,4 +62,5 @@ unsigned char *srUtilStrDup(unsigned char *pOld, size_t len); int makeFileParentDirs(uchar *szFile, size_t lenFile, mode_t mode, uid_t uid, gid_t gid, int bFailOnChown); int execProg(uchar *program, int wait, uchar *arg); +void skipWhiteSpace(uchar **pp); #endif diff --git a/syslogd.c b/syslogd.c index 8ee890f1..49d97cb3 100644 --- a/syslogd.c +++ b/syslogd.c @@ -149,7 +149,6 @@ #include #include #include -#include #include #include @@ -213,6 +212,7 @@ #include "msg.h" #include "modules.h" #include "tcpsyslog.h" +#include "cfsysline.h" #include "omshell.h" #include "omusrmsg.h" #include "ommysql.h" @@ -3432,25 +3432,6 @@ static rsRetVal addAllowedSenderLine(char* pName, uchar** ppRestOfConfLine) } -/* skip over whitespace in a standard C string. The - * provided pointer is advanced to the first non-whitespace - * charater or the \0 byte, if there is none. It is never - * moved past the \0. - */ -static void skipWhiteSpace(uchar **pp) -{ - register uchar *p; - - assert(pp != NULL); - assert(*pp != NULL); - - p = *pp; - while(*p && isspace((int) *p)) - ++p; - *pp = p; -} - - /* Parse and interpret a $DynaFileCacheSize line. * Parameter **pp has a pointer to the current config line. * On exit, it will be updated to the processed position. @@ -3501,61 +3482,6 @@ static void doDynaFileCacheSizeLine(uchar **pp) } -/* Parse and interpret an on/off inside a config file line. This is most - * often used for boolean options, but of course it may also be used - * for other things. The passed-in pointer is updated to point to - * the first unparsed character on exit. Function emits error messages - * if the value is neither on or off. It returns 0 if the option is off, - * 1 if it is on and another value if there was an error. - * rgerhards, 2007-07-15 - */ -static int doParseOnOffOption(uchar **pp) -{ - uchar *pOptStart; - uchar szOpt[32]; - - assert(pp != NULL); - assert(*pp != NULL); - - pOptStart = *pp; - skipWhiteSpace(pp); /* skip over any whitespace */ - - if(getSubString(pp, (char*) szOpt, sizeof(szOpt) / sizeof(uchar), ' ') != 0) { - logerror("Invalid $-configline - could not extract on/off option"); - return -1; - } - - if(!strcmp((char*)szOpt, "on")) { - return 1; - } else if(!strcmp((char*)szOpt, "off")) { - return 0; - } else { - logerrorSz("Option value must be on or off, but is '%s'", (char*)pOptStart); - return -1; - } -} - - -/* Parse and process an binary cofig option. pVal must be - * a pointer to an integer which is to receive the option - * value. - * rgerhards, 2007-07-15 - */ -static void doBinaryOptionLine(uchar **pp, int *pVal) -{ - int iOption; - - assert(pp != NULL); - assert(*pp != NULL); - assert(pVal != NULL); - - if((iOption = doParseOnOffOption(pp)) == -1) - return; /* nothing left to do */ - - *pVal = iOption; - skipWhiteSpace(pp); /* skip over any whitespace */ -} - /* process a $ModLoad config line. * As of now, it is a dummy, that will later evolve into the @@ -3619,38 +3545,6 @@ static void doGetGID(uchar **pp, gid_t *pGid) } -/* extract a username and return its uid. - * rgerhards, 2007-07-17 - */ -static void doGetUID(uchar **pp, uid_t *pUid) -{ - struct passwd *ppwBuf; - struct passwd pwBuf; - uchar szName[256]; - char stringBuf[2048]; /* I hope this is large enough... */ - - assert(pp != NULL); - assert(*pp != NULL); - assert(pUid != NULL); - - if(getSubString(pp, (char*) szName, sizeof(szName) / sizeof(uchar), ' ') != 0) { - logerror("could not extract user name"); - return; - } - - getpwnam_r((char*)szName, &pwBuf, stringBuf, sizeof(stringBuf), &ppwBuf); - - if(ppwBuf == NULL) { - logerrorSz("ID for user '%s' could not be found or error", (char*)szName); - } else { - *pUid = ppwBuf->pw_uid; - dprintf("uid %d obtained for user '%s'\n", *pUid, szName); - } - - skipWhiteSpace(pp); /* skip over any whitespace */ -} - - /* parse the control character escape prefix and store it. * added 2007-07-17 by rgerhards */ @@ -3835,31 +3729,31 @@ void cfsysline(uchar *p) } else if(!strcasecmp((char*) szCmd, "umask")) { doFileCreateModeUmaskLine(&p, DIR_UMASK); } else if(!strcasecmp((char*) szCmd, "dirowner")) { - doGetUID(&p, &dirUID); + doGetUID(&p, NULL, &dirUID); } else if(!strcasecmp((char*) szCmd, "dirgroup")) { doGetGID(&p, &dirGID); } else if(!strcasecmp((char*) szCmd, "fileowner")) { - doGetUID(&p, &fileUID); + doGetUID(&p, NULL, &fileUID); } else if(!strcasecmp((char*) szCmd, "filegroup")) { doGetGID(&p, &fileGID); } else if(!strcasecmp((char*) szCmd, "dynafilecachesize")) { doDynaFileCacheSizeLine(&p); } else if(!strcasecmp((char*) szCmd, "repeatedmsgreduction")) { - doBinaryOptionLine(&p, &bReduceRepeatMsgs); + doBinaryOptionLine(&p, NULL, &bReduceRepeatMsgs); } else if(!strcasecmp((char*) szCmd, "controlcharacterescapeprefix")) { doControlCharEscPrefix(&p); } else if(!strcasecmp((char*) szCmd, "escapecontrolcharactersonreceive")) { - doBinaryOptionLine(&p, &bEscapeCCOnRcv); + doBinaryOptionLine(&p, NULL, &bEscapeCCOnRcv); } else if(!strcasecmp((char*) szCmd, "dropmsgswithmaliciousdnsptrrecords")) { - doBinaryOptionLine(&p, &bDropMalPTRMsgs); + doBinaryOptionLine(&p, NULL, &bDropMalPTRMsgs); } else if(!strcasecmp((char*) szCmd, "createdirs")) { - doBinaryOptionLine(&p, &bCreateDirs); + doBinaryOptionLine(&p, NULL, &bCreateDirs); } else if(!strcasecmp((char*) szCmd, "debugprinttemplatelist")) { - doBinaryOptionLine(&p, &bDebugPrintTemplateList); + doBinaryOptionLine(&p, NULL, &bDebugPrintTemplateList); } else if(!strcasecmp((char*) szCmd, "failonchownfailure")) { - doBinaryOptionLine(&p, &bFailOnChown); + doBinaryOptionLine(&p, NULL, &bFailOnChown); } else if(!strcasecmp((char*) szCmd, "droptrailinglfonreception")) { - doBinaryOptionLine(&p, &bDropTrailingLF); + doBinaryOptionLine(&p, NULL, &bDropTrailingLF); } else if(!strcasecmp((char*) szCmd, "resetconfigvariables")) { resetConfigVariables(); } else if(!strcasecmp((char*) szCmd, "modload")) { -- cgit