[Bacula-bugs] [bacula 0000388]: Time recognition in daemon configuration is broken (and I'm afraid that in 1.37/1.38 too)

[<bac...@li...>]

The following bug has been CLOSED
======================================================================
http://bugs.bacula.org/bug_view_advanced_page.php?bug_id=0000388
======================================================================
Reported By:                Rudolf Cejka
Assigned To:                
======================================================================
Project:                    bacula
Bug ID:                     388
Category:                   Director
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
======================================================================
Date Submitted:             07-28-2005 04:53 PDT
Last Modified:              07-28-2005 07:06 PDT
======================================================================
Summary:                    Time recognition in daemon configuration is broken (and I'm afraid that in 1.37/1.38 too)
Description: 
When time like "1 week 2 days 3 hours 10 mins" is specified, it is
incorrectly parsed as "1 week 2 seconds", because store_time() in
parse_conf.c reads just first number "1" and first identifier/string "week
2" and duration_to_utime("1week2") understands it as "1 week 2 seconds". I
have created primitive patch, which reads all time values and modifiers
(if is changed to while), but I'm not sure, what is broken more: If
store_time(), or lex_get_token(), because the behavior, that "week 2" is
recognized as lex_string, where first space between "k" and "2" is
included (lexer is in lex_identifier state, where spaces are skipped), but
the second space between "2" and "d" is rejected (lexer is already in
lex_string state after "2" is found), gets me totally crazy :o)))
======================================================================

----------------------------------------------------------------------
 kern - 07-28-2005 07:05 PDT 
----------------------------------------------------------------------
Thanks for the correction.

----------------------------------------------------------------------
 kern - 07-28-2005 07:06 PDT 
----------------------------------------------------------------------
Fix applied.

Bug History
Date Modified  Username       Field                    Change              
======================================================================
07-28-05 04:53 Rudolf Cejka   New Bug                                      
07-28-05 04:53 Rudolf Cejka   File Added: patch-src.lib.parse_conf.c                    
07-28-05 07:05 kern           Bugnote Added: 0001087                       
07-28-05 07:06 kern           Bugnote Added: 0001088                       
07-28-05 07:06 kern           Resolution               open => fixed       
07-28-05 07:06 kern           Status                   new => closed       
======================================================================