Re: [syslog-ng] Config parse error in syslog-ng 2.0.0

27 Nov 2006

On Sun, 2006-11-26 at 13:15 +0100, Jacek Kalinski wrote:
...
Hi,
I have a config file working fine with syslog-ng-1.6.11.
But the same config won't work with 2.0.0.
Problem (as I've checked, compiling syslog-ng with YYDEBUG) is in:
dir_owner(0); dir_group(0); owner(0); group(0);
2 first lines of config:
options { dir_perm(0700); perm(0600); chain_hostnames(no); stats(1200);
          keep_hostname(yes); dir_owner(0); dir_group(0); owner(0);
group(0); };
Debug from checking syntax:
...
Reducing stack by rule 212 (line 731), KW_STATS_FREQ '(' NUMBER ')' ->
options_item
Stack now 0 4 18 73 156 73 156 73 156
Entering state 73
Reading a token: Next token is token ';' ()
Shifting token ';' ()
Entering state 156
Reading a token: Next token is token KW_KEEP_HOSTNAME ()
Shifting token KW_KEEP_HOSTNAME ()
Entering state 42
Reading a token: Next token is token '(' ()
Shifting token '(' ()
Entering state 125
Reading a token: Next token is token KW_YES ()
Shifting token KW_YES ()
Entering state 206
Reducing stack by rule 271 (line 868), KW_YES -> yesno
Stack now 0 4 18 73 156 73 156 73 156 73 156 42 125
Entering state 211
Reading a token: Next token is token ')' ()
Shifting token ')' ()
Entering state 308
Reducing stack by rule 217 (line 736), KW_KEEP_HOSTNAME '(' yesno ')' ->
options_item
Stack now 0 4 18 73 156 73 156 73 156 73 156
Entering state 73
Reading a token: Next token is token ';' ()
Shifting token ';' ()
Entering state 156
Reading a token: Next token is token KW_DIR_OWNER ()
Shifting token KW_DIR_OWNER ()
Entering state 66
Reading a token: Next token is token '(' ()
Shifting token '(' ()
Entering state 149
Reading a token: Next token is token NUMBER ()
syntax error at 2
Error: popping token '(' ()
Stack now 0 4 18 73 156 73 156 73 156 73 156 73 156 66
Error: popping token KW_DIR_OWNER ()
Stack now 0 4 18 73 156 73 156 73 156 73 156 73 156
Error: popping token ';' ()
Stack now 0 4 18 73 156 73 156 73 156 73 156 73
Error: popping nterm options_item ()
Stack now 0 4 18 73 156 73 156 73 156 73 156
Error: popping token ';' ()
Stack now 0 4 18 73 156 73 156 73 156 73
Error: popping nterm options_item ()
Stack now 0 4 18 73 156 73 156 73 156
Error: popping token ';' ()
Stack now 0 4 18 73 156 73 156 73
Error: popping nterm options_item ()
Stack now 0 4 18 73 156 73 156
Error: popping token ';' ()
Stack now 0 4 18 73 156 73
Error: popping nterm options_item ()
Stack now 0 4 18 73 156
Error: popping token ';' ()
Stack now 0 4 18 73
Error: popping nterm options_item ()
Stack now 0 4 18
Error: popping token '{' ()
Stack now 0 4
Error: popping token KW_OPTIONS ()
Stack now 0
Cleanup: discarding lookahead token NUMBER ()
When I change 'dir_owner' to:
dir_owner(root)
next error found is on 'dir_group':
Reading a token: Next token is token KW_DIR_GROUP ()
Shifting token KW_DIR_GROUP ()
Entering state 67
Reading a token: Next token is token '(' ()
Shifting token '(' ()
Entering state 150
Reading a token: Next token is token NUMBER ()
syntax error at 2
BUT manual, says:
       dir_owner(uid)
              User id.
documantecion says:
â                       â            â              âThe default owner
of     â
âdir_owner()            âuserid      âroot          ânewly
created            â
â                       â            â             
âdirectories.             â
The same problem with: owner and group options (so all options working
with UID/GID).
Is it an error in my configuration, or in docs? :)
Is it possible to keep compatibility with syslog-1.6 (so use UID/GID
instead of username)?
Can you check if this patch should also be ok)

--- orig/src/cfg-grammar.y +++ mod/src/cfg-grammar.y @@ -183,6 +183,7 @@ gint last_addr_family %type   <num> dnsmode

%type  <cptr> string +%type  <cptr> string_or_number

%%

@@ -339,8 +340,8 @@ source_afunix_op ;

source_afunix_option -       : KW_OWNER '(' string ')' -       | KW_GROUP '(' string ')' +       : KW_OWNER '(' string_or_number ')' +       | KW_GROUP '(' string_or_number ')' | KW_PERM '(' NUMBER ')' | KW_OPTIONAL '(' yesno ')' | source_afsocket_stream_params @@ -494,11 +495,11 @@ dest_affile_op | KW_COMPRESS '(' yesno ')' | KW_ENCRYPT '(' yesno ')' */ -       | KW_OWNER '(' string ')' -       | KW_GROUP '(' string ')' +       | KW_OWNER '(' string_or_number ')' +       | KW_GROUP '(' string_or_number ')' | KW_PERM '(' NUMBER ')' -       | KW_DIR_OWNER '(' string ')' -       | KW_DIR_GROUP '(' string ')' +       | KW_DIR_OWNER '(' +       | KW_DIR_GROUP '(' | KW_DIR_PERM '(' NUMBER ')' | KW_CREATE_DIRS '(' yesno ')' | KW_REMOVE_IF_OLDER '(' NUMBER ')' @@ -526,8 +527,8 @@ dest_afpipe_options

dest_afpipe_option : dest_writer_option -       | KW_OWNER '(' string ')' -       | KW_GROUP '(' string ')' +       | KW_OWNER '(' string_or_number ')' +       | KW_GROUP '(' string_or_number ')' | KW_PERM '(' NUMBER ')' ;

@@ -760,11 +761,11 @@ options_item | KW_GC_BUSY_THRESHOLD '(' NUMBER ')' | KW_GC_IDLE_THRESHOLD '(' NUMBER ')' | KW_CREATE_DIRS '(' yesno ')' -       | KW_OWNER '(' string ')' -       | KW_GROUP '(' string ')' +       | KW_OWNER '(' string_or_number ')' +       | KW_GROUP '(' string_or_number ')' | KW_PERM '(' NUMBER ')' -       | KW_DIR_OWNER '(' string ')' -       | KW_DIR_GROUP '(' string ')' +       | KW_DIR_OWNER '(' +       | KW_DIR_GROUP '(' | KW_DIR_PERM '(' NUMBER ')' | KW_DNS_CACHE '(' yesno ')' | KW_DNS_CACHE_SIZE '(' NUMBER ')' @@ -881,6 +882,10 @@ string | STRING ;

+string_or_number +        : STRING +        | NUMBER + %%

extern int linenum;

-- Bazsi

fixes this problem? (tomorrow's snapshot = AF_INET; tions { afunix_sd_set_uid(last_driver, $3); free($3); } { afunix_sd_set_gid(last_driver, $3); free($3); } { afunix_sd_set_uid(last_driver, $3); free($3); } { afunix_sd_set_gid(last_driver, $3); free($3); } { afunix_sd_set_perm(last_driver, $3); } { last_driver->optional = $3; } {} tion { affile_dd_set_compress(last_driver, $3); } { affile_dd_set_encrypt(last_driver, $3); } { affile_dd_set_file_uid(last_driver, $3); free($3); } { affile_dd_set_file_gid(last_driver, $3); free($3); } { affile_dd_set_file_uid(last_driver, $3); free($3); } { affile_dd_set_file_gid(last_driver, $3); free($3); } { affile_dd_set_file_perm(last_driver, $3); } { affile_dd_set_dir_uid(last_driver, $3); free($3); } { affile_dd_set_dir_gid(last_driver, $3); free($3); } string_or_number ')' { affile_dd_set_dir_uid(last_driver, $3); free($3); } string_or_number ')' { affile_dd_set_dir_gid(last_driver, $3); free($3); } { affile_dd_set_dir_perm(last_driver, $3); } { affile_dd_set_create_dirs(last_driver, $3); } { affile_dd_set_remove_if_older(last_driver, $3); } { affile_dd_set_file_uid(last_driver, $3); free($3); } { affile_dd_set_file_gid(last_driver, $3); free($3); } { affile_dd_set_file_uid(last_driver, $3); free($3); } { affile_dd_set_file_gid(last_driver, $3); free($3); } { affile_dd_set_file_perm(last_driver, $3); } { /* ignored */; } { /* ignored */; } { configuration->create_dirs = $3; } { cfg_file_owner_set(configuration, $3); free($3); } { cfg_file_group_set(configuration, $3); free($3); } { cfg_file_owner_set(configuration, $3); free($3); } { cfg_file_group_set(configuration, $3); free($3); } { cfg_file_perm_set(configuration, $3); } { cfg_dir_owner_set(configuration, $3); free($3); } { cfg_dir_group_set(configuration, $3); free($3); } string_or_number ')' { cfg_dir_owner_set(configuration, $3); free($3); } string_or_number ')' { cfg_dir_group_set(configuration, $3); free($3); } { cfg_dir_perm_set(configuration, $3); } { configuration->use_dns_cache = $3; } { configuration->dns_cache_size = $3; } { $$ = $1; } { char buf[16]; snprintf(buf, sizeof(buf), "%d", $1); $$ = strdup(buf); }

    

Re: [syslog-ng] Config parse error in syslog-ng 2.0.0

Balazs Scheidler