]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/dird/dird_conf.c
Fix hourly scheduling problem
[bacula/bacula] / bacula / src / dird / dird_conf.c
index fd34100dbb0c475b8606a8f1dd2749a66e421776..491212b0f65b2e9f57cd49ef635a5da2ee6f3c43 100644 (file)
@@ -1,28 +1,28 @@
 /*
  *   Main configuration file parser for Bacula Directors,
  *    some parts may be split into separate files such as
- *    the schedule configuration (sch_config.c).
+ *    the schedule configuration (run_config.c).
  *
  *   Note, the configuration file parser consists of three parts
  *
  *   1. The generic lexical scanner in lib/lex.c and lib/lex.h
  *
  *   2. The generic config  scanner in lib/parse_config.c and 
- *      lib/parse_config.h.
- *      These files contain the parser code, some utility
- *      routines, and the common store routines (name, int,
- *      string).
+ *     lib/parse_config.h.
+ *     These files contain the parser code, some utility
+ *     routines, and the common store routines (name, int,
+ *     string).
  *
  *   3. The daemon specific file, which contains the Resource
- *      definitions as well as any specific store routines
- *      for the resource records.
+ *     definitions as well as any specific store routines
+ *     for the resource records.
  *
  *     Kern Sibbald, January MM
  *
  *     Version $Id$
  */
 /*
-   Copyright (C) 2000-2003 Kern Sibbald and John Walker
+   Copyright (C) 2000-2004 Kern Sibbald and John Walker
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License as
  */
 int r_first = R_FIRST;
 int r_last  = R_LAST;
-pthread_mutex_t res_mutex =  PTHREAD_MUTEX_INITIALIZER;
 
 /* Imported subroutines */
-extern void store_run(LEX *lc, struct res_items *item, int index, int pass);
-extern void store_finc(LEX *lc, struct res_items *item, int index, int pass);
-extern void store_inc(LEX *lc, struct res_items *item, int index, int pass);
+extern void store_run(LEX *lc, RES_ITEM *item, int index, int pass);
+extern void store_finc(LEX *lc, RES_ITEM *item, int index, int pass);
+extern void store_inc(LEX *lc, RES_ITEM *item, int index, int pass);
 
 
 /* Forward referenced subroutines */
 
-static void store_backup(LEX *lc, struct res_items *item, int index, int pass);
-static void store_restore(LEX *lc, struct res_items *item, int index, int pass);
-static void store_jobtype(LEX *lc, struct res_items *item, int index, int pass);
-static void store_level(LEX *lc, struct res_items *item, int index, int pass);
-static void store_replace(LEX *lc, struct res_items *item, int index, int pass);
+void store_jobtype(LEX *lc, RES_ITEM *item, int index, int pass);
+void store_level(LEX *lc, RES_ITEM *item, int index, int pass);
+void store_replace(LEX *lc, RES_ITEM *item, int index, int pass);
+void store_acl(LEX *lc, RES_ITEM *item, int index, int pass);
 
 
 /* We build the current resource here as we are
@@ -83,9 +81,9 @@ int  res_all_size = sizeof(res_all);
 /* 
  *    Director Resource
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
-static struct res_items dir_items[] = {
+static RES_ITEM dir_items[] = {
    {"name",        store_name,     ITEM(res_dir.hdr.name), 0, ITEM_REQUIRED, 0},
    {"description", store_str,      ITEM(res_dir.hdr.desc), 0, 0, 0},
    {"messages",    store_res,      ITEM(res_dir.messages), R_MSGS, 0, 0},
@@ -107,13 +105,22 @@ static struct res_items dir_items[] = {
 /* 
  *    Console Resource
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
-static struct res_items con_items[] = {
+static RES_ITEM con_items[] = {
    {"name",        store_name,     ITEM(res_con.hdr.name), 0, ITEM_REQUIRED, 0},
    {"description", store_str,      ITEM(res_con.hdr.desc), 0, 0, 0},
    {"enablessl",   store_yesno,    ITEM(res_con.enable_ssl), 1, ITEM_DEFAULT, 0},
    {"password",    store_password, ITEM(res_con.password), 0, ITEM_REQUIRED, 0},
+   {"jobacl",      store_acl,      ITEM(res_con.ACL_lists), Job_ACL, 0, 0},
+   {"clientacl",   store_acl,      ITEM(res_con.ACL_lists), Client_ACL, 0, 0},
+   {"storageacl",  store_acl,      ITEM(res_con.ACL_lists), Storage_ACL, 0, 0},
+   {"scheduleacl", store_acl,      ITEM(res_con.ACL_lists), Schedule_ACL, 0, 0},
+   {"runacl",      store_acl,      ITEM(res_con.ACL_lists), Run_ACL, 0, 0},
+   {"poolacl",     store_acl,      ITEM(res_con.ACL_lists), Pool_ACL, 0, 0},
+   {"commandacl",  store_acl,      ITEM(res_con.ACL_lists), Command_ACL, 0, 0},
+   {"filesetacl",  store_acl,      ITEM(res_con.ACL_lists), FileSet_ACL, 0, 0},
+   {"catalogacl",  store_acl,      ITEM(res_con.ACL_lists), Catalog_ACL, 0, 0},
    {NULL, NULL, NULL, 0, 0, 0}
 };
 
@@ -121,15 +128,17 @@ static struct res_items con_items[] = {
 /* 
  *    Client or File daemon resource
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
 
-static struct res_items cli_items[] = {
+static RES_ITEM cli_items[] = {
    {"name",     store_name,       ITEM(res_client.hdr.name), 0, ITEM_REQUIRED, 0},
    {"description", store_str,     ITEM(res_client.hdr.desc), 0, 0, 0},
    {"address",  store_str,        ITEM(res_client.address),  0, ITEM_REQUIRED, 0},
+   {"fdaddress",  store_str,      ITEM(res_client.address),  0, 0, 0},
    {"fdport",   store_pint,       ITEM(res_client.FDport),   0, ITEM_DEFAULT, 9102},
    {"password", store_password,   ITEM(res_client.password), 0, ITEM_REQUIRED, 0},
+   {"fdpassword", store_password,   ITEM(res_client.password), 0, 0, 0},
    {"catalog",  store_res,        ITEM(res_client.catalog),  R_CATALOG, 0, 0},
    {"fileretention", store_time,  ITEM(res_client.FileRetention), 0, ITEM_DEFAULT, 60*60*24*60},
    {"jobretention",  store_time,  ITEM(res_client.JobRetention),  0, ITEM_DEFAULT, 60*60*24*180},
@@ -141,29 +150,32 @@ static struct res_items cli_items[] = {
 
 /* Storage daemon resource
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
-static struct res_items store_items[] = {
-   {"name",      store_name,     ITEM(res_store.hdr.name),   0, ITEM_REQUIRED, 0},
-   {"description", store_str,    ITEM(res_store.hdr.desc),   0, 0, 0},
-   {"sdport",    store_pint,     ITEM(res_store.SDport),     0, ITEM_DEFAULT, 9103},
-   {"sddport",   store_pint,     ITEM(res_store.SDDport),    0, 0, 0}, /* deprecated */
-   {"address",   store_str,      ITEM(res_store.address),    0, ITEM_REQUIRED, 0},
-   {"password",  store_password, ITEM(res_store.password),   0, ITEM_REQUIRED, 0},
-   {"device",    store_strname,  ITEM(res_store.dev_name),   0, ITEM_REQUIRED, 0},
-   {"mediatype", store_strname,  ITEM(res_store.media_type), 0, ITEM_REQUIRED, 0},
-   {"autochanger", store_yesno,  ITEM(res_store.autochanger), 1, ITEM_DEFAULT, 0},
-   {"enablessl", store_yesno,    ITEM(res_store.enable_ssl),  1, ITEM_DEFAULT, 0},
+static RES_ITEM store_items[] = {
+   {"name",        store_name,     ITEM(res_store.hdr.name),   0, ITEM_REQUIRED, 0},
+   {"description", store_str,      ITEM(res_store.hdr.desc),   0, 0, 0},
+   {"sdport",      store_pint,     ITEM(res_store.SDport),     0, ITEM_DEFAULT, 9103},
+   {"address",     store_str,      ITEM(res_store.address),    0, ITEM_REQUIRED, 0},
+   {"sdaddress",   store_str,      ITEM(res_store.address),    0, 0, 0},
+   {"password",    store_password, ITEM(res_store.password),   0, ITEM_REQUIRED, 0},
+   {"sdpassword",  store_password, ITEM(res_store.password),   0, 0, 0},
+   {"device",      store_strname,  ITEM(res_store.dev_name),   0, ITEM_REQUIRED, 0},
+   {"sddevicename", store_strname, ITEM(res_store.dev_name),   0, 0, 0},
+   {"mediatype",   store_strname,  ITEM(res_store.media_type), 0, ITEM_REQUIRED, 0},
+   {"autochanger", store_yesno,    ITEM(res_store.autochanger), 1, ITEM_DEFAULT, 0},
+   {"enablessl",   store_yesno,    ITEM(res_store.enable_ssl),  1, ITEM_DEFAULT, 0},
    {"maximumconcurrentjobs", store_pint, ITEM(res_store.MaxConcurrentJobs), 0, ITEM_DEFAULT, 1},
+   {"sddport", store_pint, ITEM(res_store.SDDport), 0, 0, 0}, /* deprecated */
    {NULL, NULL, NULL, 0, 0, 0} 
 };
 
 /* 
  *    Catalog Resource Directives
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
-static struct res_items cat_items[] = {
+static RES_ITEM cat_items[] = {
    {"name",     store_name,     ITEM(res_cat.hdr.name),    0, ITEM_REQUIRED, 0},
    {"description", store_str,   ITEM(res_cat.hdr.desc),    0, 0, 0},
    {"address",  store_str,      ITEM(res_cat.db_address),  0, 0, 0},
@@ -171,6 +183,7 @@ static struct res_items cat_items[] = {
    {"dbport",   store_pint,     ITEM(res_cat.db_port),      0, 0, 0},
    /* keep this password as store_str for the moment */
    {"password", store_str,      ITEM(res_cat.db_password), 0, 0, 0},
+   {"dbpassword", store_str,    ITEM(res_cat.db_password), 0, 0, 0},
    {"user",     store_str,      ITEM(res_cat.db_user),     0, 0, 0},
    {"dbname",   store_str,      ITEM(res_cat.db_name),     0, ITEM_REQUIRED, 0},
    {"dbsocket", store_str,      ITEM(res_cat.db_socket),   0, 0, 0}, 
@@ -180,83 +193,80 @@ static struct res_items cat_items[] = {
 /* 
  *    Job Resource Directives
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
-static struct res_items job_items[] = {
-   {"name",     store_name,    ITEM(res_job.hdr.name), 0, ITEM_REQUIRED, 0},
-   {"description", store_str,  ITEM(res_job.hdr.desc), 0, 0, 0},
-   {"backup",   store_backup,  ITEM(res_job),          JT_BACKUP, 0, 0},
-   {"verify",   store_backup,  ITEM(res_job),          JT_VERIFY, 0, 0},
-   {"restore",  store_restore, ITEM(res_job),          JT_RESTORE, 0, 0},
-   {"schedule", store_res,     ITEM(res_job.schedule), R_SCHEDULE, 0, 0},
-   {"type",     store_jobtype, ITEM(res_job),          0, 0, 0},
-   {"level",    store_level,   ITEM(res_job),          0, 0, 0},
-   {"messages", store_res,     ITEM(res_job.messages), R_MSGS, 0, 0},
-   {"storage",  store_res,     ITEM(res_job.storage),  R_STORAGE, 0, 0},
-   {"pool",     store_res,     ITEM(res_job.pool),     R_POOL, 0, 0},
-   {"client",   store_res,     ITEM(res_job.client),   R_CLIENT, 0, 0},
-   {"fileset",  store_res,     ITEM(res_job.fileset),  R_FILESET, 0, 0},
-   {"where",    store_dir,     ITEM(res_job.RestoreWhere), 0, 0, 0},
-   {"replace",  store_replace, ITEM(res_job.replace), 0, ITEM_DEFAULT, REPLACE_ALWAYS},
-   {"bootstrap",store_dir,     ITEM(res_job.RestoreBootstrap), 0, 0, 0},
-   {"maxruntime", store_time,  ITEM(res_job.MaxRunTime), 0, 0, 0},
-   {"maxstartdelay", store_time,ITEM(res_job.MaxStartDelay), 0, 0, 0},
+RES_ITEM job_items[] = {
+   {"name",      store_name,    ITEM(res_job.hdr.name), 0, ITEM_REQUIRED, 0},
+   {"description", store_str,   ITEM(res_job.hdr.desc), 0, 0, 0},
+   {"type",      store_jobtype, ITEM(res_job.JobType),  0, ITEM_REQUIRED, 0},
+   {"level",     store_level,   ITEM(res_job.level),    0, 0, 0},
+   {"messages",  store_res,     ITEM(res_job.messages), R_MSGS, ITEM_REQUIRED, 0},
+   {"storage",   store_res,     ITEM(res_job.storage),  R_STORAGE, ITEM_REQUIRED, 0},
+   {"pool",      store_res,     ITEM(res_job.pool),     R_POOL, ITEM_REQUIRED, 0},
+   {"fullbackuppool",  store_res,     ITEM(res_job.full_pool),   R_POOL, 0, 0},
+   {"incrementalbackuppool",  store_res, ITEM(res_job.inc_pool), R_POOL, 0, 0},
+   {"differentialbackuppool", store_res, ITEM(res_job.dif_pool), R_POOL, 0, 0},
+   {"client",    store_res,     ITEM(res_job.client),   R_CLIENT, ITEM_REQUIRED, 0},
+   {"fileset",   store_res,     ITEM(res_job.fileset),  R_FILESET, ITEM_REQUIRED, 0},
+   {"schedule",  store_res,     ITEM(res_job.schedule), R_SCHEDULE, 0, 0},
+   {"verifyjob", store_res,     ITEM(res_job.verify_job), R_JOB, 0, 0},
+   {"jobdefs",   store_res,     ITEM(res_job.jobdefs),  R_JOBDEFS, 0, 0},
+   {"where",    store_dir,      ITEM(res_job.RestoreWhere), 0, 0, 0},
+   {"bootstrap",store_dir,      ITEM(res_job.RestoreBootstrap), 0, 0, 0},
+   {"writebootstrap",store_dir, ITEM(res_job.WriteBootstrap), 0, 0, 0},
+   {"replace",  store_replace,  ITEM(res_job.replace), 0, ITEM_DEFAULT, REPLACE_ALWAYS},
+   {"maxruntime",   store_time, ITEM(res_job.MaxRunTime), 0, 0, 0},
+   {"maxwaittime",  store_time, ITEM(res_job.MaxWaitTime), 0, 0, 0},
+   {"maxstartdelay",store_time, ITEM(res_job.MaxStartDelay), 0, 0, 0},
+   {"jobretention", store_time, ITEM(res_job.JobRetention),  0, 0, 0},
    {"prefixlinks", store_yesno, ITEM(res_job.PrefixLinks), 1, ITEM_DEFAULT, 0},
    {"prunejobs",   store_yesno, ITEM(res_job.PruneJobs), 1, ITEM_DEFAULT, 0},
    {"prunefiles",  store_yesno, ITEM(res_job.PruneFiles), 1, ITEM_DEFAULT, 0},
-   {"prunevolumes", store_yesno, ITEM(res_job.PruneVolumes), 1, ITEM_DEFAULT, 0},
+   {"prunevolumes",store_yesno, ITEM(res_job.PruneVolumes), 1, ITEM_DEFAULT, 0},
+   {"spoolattributes",store_yesno, ITEM(res_job.SpoolAttributes), 1, ITEM_DEFAULT, 0},
+   {"spooldata",   store_yesno, ITEM(res_job.spool_data), 1, ITEM_DEFAULT, 0},
    {"runbeforejob", store_str,  ITEM(res_job.RunBeforeJob), 0, 0, 0},
    {"runafterjob",  store_str,  ITEM(res_job.RunAfterJob),  0, 0, 0},
-   {"spoolattributes", store_yesno, ITEM(res_job.SpoolAttributes), 1, ITEM_DEFAULT, 0},
-   {"writebootstrap", store_dir, ITEM(res_job.WriteBootstrap), 0, 0, 0},
+   {"runafterfailedjob",  store_str,  ITEM(res_job.RunAfterFailedJob),  0, 0, 0},
+   {"clientrunbeforejob", store_str,  ITEM(res_job.ClientRunBeforeJob), 0, 0, 0},
+   {"clientrunafterjob",  store_str,  ITEM(res_job.ClientRunAfterJob),  0, 0, 0},
    {"maximumconcurrentjobs", store_pint, ITEM(res_job.MaxConcurrentJobs), 0, ITEM_DEFAULT, 1},
    {"rescheduleonerror", store_yesno, ITEM(res_job.RescheduleOnError), 1, ITEM_DEFAULT, 0},
    {"rescheduleinterval", store_time, ITEM(res_job.RescheduleInterval), 0, ITEM_DEFAULT, 60 * 30},
    {"rescheduletimes", store_pint, ITEM(res_job.RescheduleTimes), 0, 0, 0},
+   {"priority",   store_pint, ITEM(res_job.Priority), 0, ITEM_DEFAULT, 10},
    {NULL, NULL, NULL, 0, 0, 0} 
 };
 
 /* FileSet resource
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
-static struct res_items fs_items[] = {
+static RES_ITEM fs_items[] = {
    {"name",        store_name, ITEM(res_fs.hdr.name), 0, ITEM_REQUIRED, 0},
    {"description", store_str,  ITEM(res_fs.hdr.desc), 0, 0, 0},
-   {"include",     store_inc,  NULL,                  0, 0, 0},
-   {"finclude",    store_finc, NULL,                  0, ITEM_NO_EQUALS, 0},
-   {"exclude",     store_inc,  NULL,                  1, 0, 0},
-   {"fexclude",    store_finc, NULL,                  1, ITEM_NO_EQUALS, 0},
-   {NULL,          NULL,       NULL,                  0, 0, 0} 
+   {"include",     store_inc,  NULL,                  0, ITEM_NO_EQUALS, 0},
+   {"exclude",     store_inc,  NULL,                  1, ITEM_NO_EQUALS, 0},
+   {NULL,         NULL,       NULL,                  0, 0, 0} 
 };
 
 /* Schedule -- see run_conf.c */
 /* Schedule
  *
- *   name          handler     value                 code flags    default_value
+ *   name         handler     value                 code flags    default_value
  */
-static struct res_items sch_items[] = {
+static RES_ITEM sch_items[] = {
    {"name",     store_name,  ITEM(res_sch.hdr.name), 0, ITEM_REQUIRED, 0},
    {"description", store_str, ITEM(res_sch.hdr.desc), 0, 0, 0},
    {"run",      store_run,   ITEM(res_sch.run),      0, 0, 0},
    {NULL, NULL, NULL, 0, 0, 0} 
 };
 
-/* Group resource -- not implemented
- *
- *   name          handler     value                 code flags    default_value
- */
-static struct res_items group_items[] = {
-   {"name",        store_name, ITEM(res_group.hdr.name), 0, ITEM_REQUIRED, 0},
-   {"description", store_str,  ITEM(res_group.hdr.desc), 0, 0, 0},
-   {NULL, NULL, NULL, 0, 0, 0} 
-};
-
 /* Pool resource
  *
- *   name             handler     value                        code flags default_value
+ *   name            handler     value                        code flags default_value
  */
-static struct res_items pool_items[] = {
+static RES_ITEM pool_items[] = {
    {"name",            store_name,    ITEM(res_pool.hdr.name),      0, ITEM_REQUIRED, 0},
    {"description",     store_str,     ITEM(res_pool.hdr.desc),      0, 0,     0},
    {"pooltype",        store_strname, ITEM(res_pool.pool_type),     0, ITEM_REQUIRED, 0},
@@ -264,7 +274,9 @@ static struct res_items pool_items[] = {
    {"cleaningprefix",  store_strname, ITEM(res_pool.cleaning_prefix),  0, 0,     0},
    {"usecatalog",      store_yesno, ITEM(res_pool.use_catalog),     1, ITEM_DEFAULT,  1},
    {"usevolumeonce",   store_yesno, ITEM(res_pool.use_volume_once), 1, 0,        0},
+   {"purgeoldestvolume", store_yesno, ITEM(res_pool.purge_oldest_volume), 1, 0, 0},
    {"recycleoldestvolume", store_yesno, ITEM(res_pool.recycle_oldest_volume), 1, 0, 0},
+   {"recyclecurrentvolume", store_yesno, ITEM(res_pool.recycle_current_volume), 1, 0, 0},
    {"maximumvolumes",  store_pint,  ITEM(res_pool.max_volumes),     0, 0,        0},
    {"maximumvolumejobs", store_pint,  ITEM(res_pool.MaxVolJobs),    0, 0,       0},
    {"maximumvolumefiles", store_pint, ITEM(res_pool.MaxVolFiles),   0, 0,       0},
@@ -280,9 +292,9 @@ static struct res_items pool_items[] = {
 
 /* 
  * Counter Resource
- *   name             handler     value                        code flags default_value
+ *   name            handler     value                        code flags default_value
  */
-static struct res_items counter_items[] = {
+static RES_ITEM counter_items[] = {
    {"name",            store_name,    ITEM(res_counter.hdr.name),        0, ITEM_REQUIRED, 0},
    {"description",     store_str,     ITEM(res_counter.hdr.desc),        0, 0,     0},
    {"minimum",         store_int,     ITEM(res_counter.MinValue),        0, ITEM_DEFAULT, 0},
@@ -294,34 +306,37 @@ static struct res_items counter_items[] = {
 
 
 /* Message resource */
-extern struct res_items msgs_items[];
+extern RES_ITEM msgs_items[];
 
 /* 
  * This is the master resource definition.  
  * It must have one item for each of the resources.
  *
- *  name             items        rcode        res_head
+ *  NOTE!!! keep it in the same order as the R_codes
+ *    or eliminate all resources[rindex].name
+ *
+ *  name            items        rcode        res_head
  */
-struct s_res resources[] = {
+RES_TABLE resources[] = {
    {"director",      dir_items,   R_DIRECTOR,  NULL},
-   {"console",       con_items,   R_CONSOLE,   NULL},
    {"client",        cli_items,   R_CLIENT,    NULL},
    {"job",           job_items,   R_JOB,       NULL},
    {"storage",       store_items, R_STORAGE,   NULL},
    {"catalog",       cat_items,   R_CATALOG,   NULL},
    {"schedule",      sch_items,   R_SCHEDULE,  NULL},
    {"fileset",       fs_items,    R_FILESET,   NULL},
-   {"group",         group_items, R_GROUP,     NULL},
    {"pool",          pool_items,  R_POOL,      NULL},
    {"messages",      msgs_items,  R_MSGS,      NULL},
    {"counter",       counter_items, R_COUNTER, NULL},
-   {NULL,            NULL,        0,           NULL}
+   {"console",       con_items,   R_CONSOLE,   NULL},
+   {"jobdefs",       job_items,   R_JOBDEFS,   NULL},
+   {NULL,           NULL,        0,           NULL}
 };
 
 
 /* Keywords (RHS) permitted in Job Level records   
  *
- *   level_name      level              job_type
+ *   level_name      level             job_type
  */
 struct s_jl joblevels[] = {
    {"Full",          L_FULL,            JT_BACKUP},
@@ -330,31 +345,35 @@ struct s_jl joblevels[] = {
    {"Differential",  L_DIFFERENTIAL,    JT_BACKUP},
    {"Since",         L_SINCE,           JT_BACKUP},
    {"Catalog",       L_VERIFY_CATALOG,  JT_VERIFY},
-   {"Initcatalog",   L_VERIFY_INIT,     JT_VERIFY},
+   {"InitCatalog",   L_VERIFY_INIT,     JT_VERIFY},
    {"VolumeToCatalog", L_VERIFY_VOLUME_TO_CATALOG,   JT_VERIFY},
+   {"DiskToCatalog", L_VERIFY_DISK_TO_CATALOG,   JT_VERIFY},
    {"Data",          L_VERIFY_DATA,     JT_VERIFY},
-   {NULL,            0}
+   {" ",             L_NONE,            JT_ADMIN},
+   {" ",             L_NONE,            JT_RESTORE},
+   {NULL,           0,                          0}
 };
 
 /* Keywords (RHS) permitted in Job type records   
  *
- *   type_name       job_type
+ *   type_name      job_type
  */
 struct s_jt jobtypes[] = {
    {"backup",        JT_BACKUP},
    {"admin",         JT_ADMIN},
    {"verify",        JT_VERIFY},
    {"restore",       JT_RESTORE},
-   {NULL,            0}
+   {NULL,           0}
 };
 
+#ifdef old_deprecated_code
 
 /* Keywords (RHS) permitted in Backup and Verify records */
 static struct s_kw BakVerFields[] = {
    {"client",        'C'},
    {"fileset",       'F'},
    {"level",         'L'}, 
-   {NULL,            0}
+   {NULL,           0}
 };
 
 /* Keywords (RHS) permitted in Restore records */
@@ -365,8 +384,9 @@ static struct s_kw RestoreFields[] = {
    {"where",         'W'},            /* root of restore */
    {"replace",       'R'},            /* replacement options */
    {"bootstrap",     'B'},            /* bootstrap file */
-   {NULL,              0}
+   {NULL,             0}
 };
+#endif
 
 /* Options permitted in Restore replace= */
 struct s_kw ReplaceOptions[] = {
@@ -374,7 +394,7 @@ struct s_kw ReplaceOptions[] = {
    {"ifnewer",        REPLACE_IFNEWER},
    {"ifolder",        REPLACE_IFOLDER},
    {"never",          REPLACE_NEVER},
-   {NULL,               0}
+   {NULL,              0}
 };
 
 char *level_to_str(int level)
@@ -383,11 +403,11 @@ char *level_to_str(int level)
    static char level_no[30];
    char *str = level_no;
 
-   sprintf(level_no, "%d", level);    /* default if not found */
+   bsnprintf(level_no, sizeof(level_no), "%d", level);    /* default if not found */
    for (i=0; joblevels[i].level_name; i++) {
       if (level == joblevels[i].level) {
-         str = joblevels[i].level_name;
-         break;
+        str = joblevels[i].level_name;
+        break;
       }
    }
    return str;
@@ -397,94 +417,100 @@ char *level_to_str(int level)
 void dump_resource(int type, RES *reshdr, void sendit(void *sock, char *fmt, ...), void *sock)
 {
    URES *res = (URES *)reshdr;
-   int recurse = 1;
-   char ed1[30], ed2[30];
+   bool recurse = true;
+   char ed1[100], ed2[100];
 
    if (res == NULL) {
       sendit(sock, "No %s resource defined\n", res_to_str(type));
       return;
    }
-   if (type < 0) {                    /* no recursion */
+   if (type < 0) {                   /* no recursion */
       type = - type;
-      recurse = 0;
+      recurse = false;
    }
    switch (type) {
    case R_DIRECTOR:
       sendit(sock, "Director: name=%s MaxJobs=%d FDtimeout=%s SDtimeout=%s\n", 
-         reshdr->name, res->res_dir.MaxConcurrentJobs, 
-         edit_uint64(res->res_dir.FDConnectTimeout, ed1),
-         edit_uint64(res->res_dir.SDConnectTimeout, ed2));
+        reshdr->name, res->res_dir.MaxConcurrentJobs, 
+        edit_uint64(res->res_dir.FDConnectTimeout, ed1),
+        edit_uint64(res->res_dir.SDConnectTimeout, ed2));
       if (res->res_dir.query_file) {
          sendit(sock, "   query_file=%s\n", res->res_dir.query_file);
       }
       if (res->res_dir.messages) {
          sendit(sock, "  --> ");
-         dump_resource(-R_MSGS, (RES *)res->res_dir.messages, sendit, sock);
+        dump_resource(-R_MSGS, (RES *)res->res_dir.messages, sendit, sock);
       }
       break;
    case R_CONSOLE:
       sendit(sock, "Console: name=%s SSL=%d\n", 
-         res->res_con.hdr.name, res->res_con.enable_ssl);
+        res->res_con.hdr.name, res->res_con.enable_ssl);
       break;
    case R_COUNTER:
-      sendit(sock, "Counter: name=%s min=%d max=%d\n",
-         res->res_counter.hdr.name, res->res_counter.MinValue, 
-         res->res_counter.MaxValue);
-      if (res->res_counter.Catalog) {
-         sendit(sock, "  --> ");
-         dump_resource(-R_CATALOG, (RES *)res->res_counter.Catalog, sendit, sock);
-      }
       if (res->res_counter.WrapCounter) {
+         sendit(sock, "Counter: name=%s min=%d max=%d cur=%d wrapcntr=%s\n",
+           res->res_counter.hdr.name, res->res_counter.MinValue, 
+           res->res_counter.MaxValue, res->res_counter.CurrentValue,
+           res->res_counter.WrapCounter->hdr.name);
+      } else {
+         sendit(sock, "Counter: name=%s min=%d max=%d\n",
+           res->res_counter.hdr.name, res->res_counter.MinValue, 
+           res->res_counter.MaxValue);
+      }
+      if (res->res_counter.Catalog) {
          sendit(sock, "  --> ");
-         dump_resource(-R_COUNTER, (RES *)res->res_counter.WrapCounter, sendit, sock);
+        dump_resource(-R_CATALOG, (RES *)res->res_counter.Catalog, sendit, sock);
       }
+      break;
 
-
-
-         break;
    case R_CLIENT:
       sendit(sock, "Client: name=%s address=%s FDport=%d MaxJobs=%u\n",
-         res->res_client.hdr.name, res->res_client.address, res->res_client.FDport,
-         res->res_client.MaxConcurrentJobs);
-      sendit(sock, "      JobRetention=%" lld " FileRetention=%" lld " AutoPrune=%d\n",
-         res->res_client.JobRetention, res->res_client.FileRetention,
-         res->res_client.AutoPrune);
+        res->res_client.hdr.name, res->res_client.address, res->res_client.FDport,
+        res->res_client.MaxConcurrentJobs);
+      sendit(sock, "      JobRetention=%s FileRetention=%s AutoPrune=%d\n",
+        edit_utime(res->res_client.JobRetention, ed1), 
+        edit_utime(res->res_client.FileRetention, ed2),
+        res->res_client.AutoPrune);
       if (res->res_client.catalog) {
          sendit(sock, "  --> ");
-         dump_resource(-R_CATALOG, (RES *)res->res_client.catalog, sendit, sock);
+        dump_resource(-R_CATALOG, (RES *)res->res_client.catalog, sendit, sock);
       }
       break;
    case R_STORAGE:
       sendit(sock, "Storage: name=%s address=%s SDport=%d MaxJobs=%u\n\
       DeviceName=%s MediaType=%s\n",
-         res->res_store.hdr.name, res->res_store.address, res->res_store.SDport,
-         res->res_store.MaxConcurrentJobs,
-         res->res_store.dev_name, res->res_store.media_type);
+        res->res_store.hdr.name, res->res_store.address, res->res_store.SDport,
+        res->res_store.MaxConcurrentJobs,
+        res->res_store.dev_name, res->res_store.media_type);
       break;
    case R_CATALOG:
       sendit(sock, "Catalog: name=%s address=%s DBport=%d db_name=%s\n\
       db_user=%s\n",
-         res->res_cat.hdr.name, NPRT(res->res_cat.db_address),
-         res->res_cat.db_port, res->res_cat.db_name, NPRT(res->res_cat.db_user));
+        res->res_cat.hdr.name, NPRT(res->res_cat.db_address),
+        res->res_cat.db_port, res->res_cat.db_name, NPRT(res->res_cat.db_user));
       break;
    case R_JOB:
-      sendit(sock, "Job: name=%s JobType=%d level=%s MaxJobs=%u\n", 
-         res->res_job.hdr.name, res->res_job.JobType, 
-         level_to_str(res->res_job.level), res->res_job.MaxConcurrentJobs);
-      sendit(sock, "     Resched=%d Times=%d Interval=%s\n",
-          res->res_job.RescheduleOnError, res->res_job.RescheduleTimes,
-          edit_uint64_with_commas(res->res_job.RescheduleInterval, ed1));
+   case R_JOBDEFS:
+      sendit(sock, "%s: name=%s JobType=%d level=%s Priority=%d MaxJobs=%u\n", 
+         type == R_JOB ? "Job" : "JobDefs",
+        res->res_job.hdr.name, res->res_job.JobType, 
+        level_to_str(res->res_job.level), res->res_job.Priority,
+        res->res_job.MaxConcurrentJobs);
+      sendit(sock, "     Resched=%d Times=%d Interval=%s Spool=%d\n",
+         res->res_job.RescheduleOnError, res->res_job.RescheduleTimes,
+         edit_uint64_with_commas(res->res_job.RescheduleInterval, ed1),
+         res->res_job.spool_data);
       if (res->res_job.client) {
          sendit(sock, "  --> ");
-         dump_resource(-R_CLIENT, (RES *)res->res_job.client, sendit, sock);
+        dump_resource(-R_CLIENT, (RES *)res->res_job.client, sendit, sock);
       }
       if (res->res_job.fileset) {
          sendit(sock, "  --> ");
-         dump_resource(-R_FILESET, (RES *)res->res_job.fileset, sendit, sock);
+        dump_resource(-R_FILESET, (RES *)res->res_job.fileset, sendit, sock);
       }
       if (res->res_job.schedule) {
          sendit(sock, "  --> ");
-         dump_resource(-R_SCHEDULE, (RES *)res->res_job.schedule, sendit, sock);
+        dump_resource(-R_SCHEDULE, (RES *)res->res_job.schedule, sendit, sock);
       }
       if (res->res_job.RestoreWhere) {
          sendit(sock, "  --> Where=%s\n", NPRT(res->res_job.RestoreWhere));
@@ -498,136 +524,186 @@ void dump_resource(int type, RES *reshdr, void sendit(void *sock, char *fmt, ...
       if (res->res_job.RunAfterJob) {
          sendit(sock, "  --> RunAfter=%s\n", NPRT(res->res_job.RunAfterJob));
       }
+      if (res->res_job.RunAfterFailedJob) {
+         sendit(sock, "  --> RunAfterFailed=%s\n", NPRT(res->res_job.RunAfterFailedJob));
+      }
       if (res->res_job.WriteBootstrap) {
          sendit(sock, "  --> WriteBootstrap=%s\n", NPRT(res->res_job.WriteBootstrap));
       }
       if (res->res_job.storage) {
          sendit(sock, "  --> ");
-         dump_resource(-R_STORAGE, (RES *)res->res_job.storage, sendit, sock);
+        dump_resource(-R_STORAGE, (RES *)res->res_job.storage, sendit, sock);
       }
       if (res->res_job.pool) {
          sendit(sock, "  --> ");
-         dump_resource(-R_POOL, (RES *)res->res_job.pool, sendit, sock);
-      } else {
-         sendit(sock, "!!! No Pool resource\n");
+        dump_resource(-R_POOL, (RES *)res->res_job.pool, sendit, sock);
+      }
+      if (res->res_job.full_pool) {
+         sendit(sock, "  --> ");
+        dump_resource(-R_POOL, (RES *)res->res_job.full_pool, sendit, sock);
+      }
+      if (res->res_job.inc_pool) {
+         sendit(sock, "  --> ");
+        dump_resource(-R_POOL, (RES *)res->res_job.inc_pool, sendit, sock);
       }
+      if (res->res_job.dif_pool) {
+         sendit(sock, "  --> ");
+        dump_resource(-R_POOL, (RES *)res->res_job.dif_pool, sendit, sock);
+      }
+      if (res->res_job.verify_job) {
+         sendit(sock, "  --> ");
+        dump_resource(-type, (RES *)res->res_job.verify_job, sendit, sock);
+      }
+      break;
       if (res->res_job.messages) {
          sendit(sock, "  --> ");
-         dump_resource(-R_MSGS, (RES *)res->res_job.messages, sendit, sock);
+        dump_resource(-R_MSGS, (RES *)res->res_job.messages, sendit, sock);
       }
       break;
    case R_FILESET:
+   {
+      int i, j, k;
       sendit(sock, "FileSet: name=%s\n", res->res_fs.hdr.name);
-      for (int i=0; i<res->res_fs.num_includes; i++) {
-         INCEXE *incexe = res->res_fs.include_items[i];
-         for (int j=0; j<incexe->num_names; j++) {
-            sendit(sock, "      Inc: %s\n", incexe->name_list[j]);
-         }
-      }
-      for (int i=0; i<res->res_fs.num_excludes; i++) {
-         INCEXE *incexe = res->res_fs.exclude_items[i];
-         for (int j=0; j<incexe->num_names; j++) {
-            sendit(sock, "      Exc: %s\n", incexe->name_list[j]);
-         }
+      for (i=0; i<res->res_fs.num_includes; i++) {
+        INCEXE *incexe = res->res_fs.include_items[i];
+        for (j=0; j<incexe->num_opts; j++) {
+           FOPTS *fo = incexe->opts_list[j];
+            sendit(sock, "      O %s\n", fo->opts);
+           for (k=0; k<fo->regex.size(); k++) {
+               sendit(sock, "      R %s\n", fo->regex.get(k));
+           }
+           for (k=0; k<fo->wild.size(); k++) {
+               sendit(sock, "      W %s\n", fo->wild.get(k));
+           }
+           for (k=0; k<fo->base.size(); k++) {
+               sendit(sock, "      B %s\n", fo->base.get(k));
+           }
+            sendit(sock, "      N\n");
+        }
+        for (j=0; j<incexe->name_list.size(); j++) {
+            sendit(sock, "      I %s\n", incexe->name_list.get(j));
+        }
+        if (incexe->name_list.size()) {
+            sendit(sock, "      N\n");
+        }
+      }
+        
+      for (i=0; i<res->res_fs.num_excludes; i++) {
+        INCEXE *incexe = res->res_fs.exclude_items[i];
+        for (j=0; j<incexe->name_list.size(); j++) {
+            sendit(sock, "      E %s\n", incexe->name_list.get(j));
+        }
+        if (incexe->name_list.size()) {
+            sendit(sock, "      N\n");
+        }
       }
       break;
+   }
    case R_SCHEDULE:
       if (res->res_sch.run) {
-         int i;
-         RUN *run = res->res_sch.run;
-         char buf[1000], num[10];
+        int i;
+        RUN *run = res->res_sch.run;
+        char buf[1000], num[30];
          sendit(sock, "Schedule: name=%s\n", res->res_sch.hdr.name);
-         if (!run) {
-            break;
-         }
+        if (!run) {
+           break;
+        }
 next_run:
          sendit(sock, "  --> Run Level=%s\n", level_to_str(run->level));
-         strcpy(buf, "      hour=");
-         for (i=0; i<24; i++) {
-            if (bit_is_set(i, run->hour)) {
-               sprintf(num, "%d ", i);
-               strcat(buf, num);
-            }
-         }
-         strcat(buf, "\n");
-         sendit(sock, buf);
-         strcpy(buf, "      mday=");
-         for (i=0; i<31; i++) {
-            if (bit_is_set(i, run->mday)) {
-               sprintf(num, "%d ", i+1);
-               strcat(buf, num);
-            }
-         }
-         strcat(buf, "\n");
-         sendit(sock, buf);
-         strcpy(buf, "      month=");
-         for (i=0; i<12; i++) {
-            if (bit_is_set(i, run->month)) {
-               sprintf(num, "%d ", i+1);
-               strcat(buf, num);
-            }
-         }
-         strcat(buf, "\n");
-         sendit(sock, buf);
-         strcpy(buf, "      wday=");
-         for (i=0; i<7; i++) {
-            if (bit_is_set(i, run->wday)) {
-               sprintf(num, "%d ", i+1);
-               strcat(buf, num);
-            }
-         }
-         strcat(buf, "\n");
-         sendit(sock, buf);
-         strcpy(buf, "      wpos=");
-         for (i=0; i<5; i++) {
-            if (bit_is_set(i, run->wpos)) {
-               sprintf(num, "%d ", i+1);
-               strcat(buf, num);
-            }
-         }
-         strcat(buf, "\n");
-         sendit(sock, buf);
+         bstrncpy(buf, "      hour=", sizeof(buf));
+        for (i=0; i<24; i++) {
+           if (bit_is_set(i, run->hour)) {
+               bsnprintf(num, sizeof(num), "%d ", i);
+              bstrncat(buf, num, sizeof(buf));
+           }
+        }
+         bstrncat(buf, "\n", sizeof(buf));
+        sendit(sock, buf);
+         bstrncpy(buf, "      mday=", sizeof(buf));
+        for (i=0; i<31; i++) {
+           if (bit_is_set(i, run->mday)) {
+               bsnprintf(num, sizeof(num), "%d ", i);
+              bstrncat(buf, num, sizeof(buf));
+           }
+        }
+         bstrncat(buf, "\n", sizeof(buf));
+        sendit(sock, buf);
+         bstrncpy(buf, "      month=", sizeof(buf));
+        for (i=0; i<12; i++) {
+           if (bit_is_set(i, run->month)) {
+               bsnprintf(num, sizeof(num), "%d ", i);
+              bstrncat(buf, num, sizeof(buf));
+           }
+        }
+         bstrncat(buf, "\n", sizeof(buf));
+        sendit(sock, buf);
+         bstrncpy(buf, "      wday=", sizeof(buf));
+        for (i=0; i<7; i++) {
+           if (bit_is_set(i, run->wday)) {
+               bsnprintf(num, sizeof(num), "%d ", i);
+              bstrncat(buf, num, sizeof(buf));
+           }
+        }
+         bstrncat(buf, "\n", sizeof(buf));
+        sendit(sock, buf);
+         bstrncpy(buf, "      wom=", sizeof(buf));
+        for (i=0; i<5; i++) {
+           if (bit_is_set(i, run->wom)) {
+               bsnprintf(num, sizeof(num), "%d ", i);
+              bstrncat(buf, num, sizeof(buf));
+           }
+        }
+         bstrncat(buf, "\n", sizeof(buf));
+        sendit(sock, buf);
+         bstrncpy(buf, "      woy=", sizeof(buf));
+        for (i=0; i<54; i++) {
+           if (bit_is_set(i, run->woy)) {
+               bsnprintf(num, sizeof(num), "%d ", i);
+              bstrncat(buf, num, sizeof(buf));
+           }
+        }
+         bstrncat(buf, "\n", sizeof(buf));
+        sendit(sock, buf);
          sendit(sock, "      mins=%d\n", run->minute);
-         if (run->pool) {
+        if (run->pool) {
             sendit(sock, "     --> ");
-            dump_resource(-R_POOL, (RES *)run->pool, sendit, sock);
-         }
-         if (run->storage) {
+           dump_resource(-R_POOL, (RES *)run->pool, sendit, sock);
+        }
+        if (run->storage) {
             sendit(sock, "     --> ");
-            dump_resource(-R_STORAGE, (RES *)run->storage, sendit, sock);
-         }
-         if (run->msgs) {
+           dump_resource(-R_STORAGE, (RES *)run->storage, sendit, sock);
+        }
+        if (run->msgs) {
             sendit(sock, "     --> ");
-            dump_resource(-R_MSGS, (RES *)run->msgs, sendit, sock);
-         }
-         /* If another Run record is chained in, go print it */
-         if (run->next) {
-            run = run->next;
-            goto next_run;
-         }
+           dump_resource(-R_MSGS, (RES *)run->msgs, sendit, sock);
+        }
+        /* If another Run record is chained in, go print it */
+        if (run->next) {
+           run = run->next;
+           goto next_run;
+        }
       } else {
          sendit(sock, "Schedule: name=%s\n", res->res_sch.hdr.name);
       }
       break;
-   case R_GROUP:
-      sendit(sock, "Group: name=%s\n", res->res_group.hdr.name);
-      break;
    case R_POOL:
       sendit(sock, "Pool: name=%s PoolType=%s\n", res->res_pool.hdr.name,
-              res->res_pool.pool_type);
+             res->res_pool.pool_type);
       sendit(sock, "      use_cat=%d use_once=%d acpt_any=%d cat_files=%d\n",
-              res->res_pool.use_catalog, res->res_pool.use_volume_once,
-              res->res_pool.accept_any_volume, res->res_pool.catalog_files);
-      sendit(sock, "      max_vols=%d auto_prune=%d VolRetention=%" lld "\n",
-              res->res_pool.max_volumes, res->res_pool.AutoPrune,
-              res->res_pool.VolRetention);
-      sendit(sock, "      recycle=%d LabelFormat=%s\n", res->res_pool.Recycle,
-              NPRT(res->res_pool.label_format));
+             res->res_pool.use_catalog, res->res_pool.use_volume_once,
+             res->res_pool.accept_any_volume, res->res_pool.catalog_files);
+      sendit(sock, "      max_vols=%d auto_prune=%d VolRetention=%s\n",
+             res->res_pool.max_volumes, res->res_pool.AutoPrune,
+             edit_utime(res->res_pool.VolRetention, ed1));
+      sendit(sock, "      VolUse=%s recycle=%d LabelFormat=%s\n", 
+             edit_utime(res->res_pool.VolUseDuration, ed1),
+             res->res_pool.Recycle,
+             NPRT(res->res_pool.label_format));
       sendit(sock, "      CleaningPrefix=%s\n",
-              NPRT(res->res_pool.cleaning_prefix));
+             NPRT(res->res_pool.cleaning_prefix));
       sendit(sock, "      recyleOldest=%d MaxVolJobs=%d MaxVolFiles=%d\n",
-              res->res_pool.recycle_oldest_volume, 
-              res->res_pool.MaxVolJobs, res->res_pool.MaxVolFiles);
+             res->res_pool.purge_oldest_volume, 
+             res->res_pool.MaxVolJobs, res->res_pool.MaxVolFiles);
       break;
    case R_MSGS:
       sendit(sock, "Messages: name=%s\n", res->res_msgs.hdr.name);
@@ -650,23 +726,12 @@ next_run:
  */
 static void free_incexe(INCEXE *incexe)
 {
-   for (int i=0; i<incexe->num_names; i++) {
-      free(incexe->name_list[i]);
-   }
-   if (incexe->name_list) {
-      free(incexe->name_list);
-   }
+   incexe->name_list.destroy();
    for (int i=0; i<incexe->num_opts; i++) {
       FOPTS *fopt = incexe->opts_list[i];
-      if (fopt->match) {
-         free(fopt->match);
-      }
-      for (int j=0; j<fopt->num_base; j++) {
-         free(fopt->base_list[j]);
-      }
-      if (fopt->base_list) {
-         free(fopt->base_list);
-      }
+      fopt->regex.destroy();
+      fopt->wild.destroy();
+      fopt->base.destroy();
       free(fopt);
    }
    if (incexe->opts_list) {
@@ -676,20 +741,17 @@ static void free_incexe(INCEXE *incexe)
 }
 
 /* 
- * Free memory of resource.  
+ * Free memory of resource -- called when daemon terminates.
  * NB, we don't need to worry about freeing any references
  * to other resources as they will be freed when that 
  * resource chain is traversed.  Mainly we worry about freeing
  * allocated strings (names).
  */
-void free_resource(int type)
+void free_resource(RES *sres, int type)
 {
    int num;
-   URES *res;
-   RES *nres;
-   int rindex = type - r_first;
-
-   res = (URES *)resources[rindex].res_head;
+   RES *nres;                        /* next resource if linked */
+   URES *res = (URES *)sres;
 
    if (res == NULL)
       return;
@@ -706,135 +768,151 @@ void free_resource(int type)
    switch (type) {
    case R_DIRECTOR:
       if (res->res_dir.working_directory) {
-         free(res->res_dir.working_directory);
+        free(res->res_dir.working_directory);
       }
       if (res->res_dir.pid_directory) {
-         free(res->res_dir.pid_directory);
+        free(res->res_dir.pid_directory);
       }
       if (res->res_dir.subsys_directory) {
-         free(res->res_dir.subsys_directory);
+        free(res->res_dir.subsys_directory);
       }
       if (res->res_dir.password) {
-         free(res->res_dir.password);
+        free(res->res_dir.password);
       }
       if (res->res_dir.query_file) {
-         free(res->res_dir.query_file);
+        free(res->res_dir.query_file);
       }
       if (res->res_dir.DIRaddr) {
-         free(res->res_dir.DIRaddr);
+        free(res->res_dir.DIRaddr);
       }
       break;
    case R_COUNTER:
        break;
    case R_CONSOLE:
       if (res->res_con.password) {
-         free(res->res_con.password);
+        free(res->res_con.password);
+      }
+      for (int i=0; i<Num_ACL; i++) {
+        if (res->res_con.ACL_lists[i]) {
+           delete res->res_con.ACL_lists[i];
+           res->res_con.ACL_lists[i] = NULL;
+        }
       }
       break;
    case R_CLIENT:
       if (res->res_client.address) {
-         free(res->res_client.address);
+        free(res->res_client.address);
       }
       if (res->res_client.password) {
-         free(res->res_client.password);
+        free(res->res_client.password);
       }
       break;
    case R_STORAGE:
       if (res->res_store.address) {
-         free(res->res_store.address);
+        free(res->res_store.address);
       }
       if (res->res_store.password) {
-         free(res->res_store.password);
+        free(res->res_store.password);
       }
       if (res->res_store.media_type) {
-         free(res->res_store.media_type);
+        free(res->res_store.media_type);
       }
       if (res->res_store.dev_name) {
-         free(res->res_store.dev_name);
+        free(res->res_store.dev_name);
       }
       break;
    case R_CATALOG:
       if (res->res_cat.db_address) {
-         free(res->res_cat.db_address);
+        free(res->res_cat.db_address);
       }
       if (res->res_cat.db_socket) {
-         free(res->res_cat.db_socket);
+        free(res->res_cat.db_socket);
       }
       if (res->res_cat.db_user) {
-         free(res->res_cat.db_user);
+        free(res->res_cat.db_user);
       }
       if (res->res_cat.db_name) {
-         free(res->res_cat.db_name);
+        free(res->res_cat.db_name);
       }
       if (res->res_cat.db_password) {
-         free(res->res_cat.db_password);
+        free(res->res_cat.db_password);
       }
       break;
    case R_FILESET:
       if ((num=res->res_fs.num_includes)) {
-         while (--num >= 0) {   
-            free_incexe(res->res_fs.include_items[num]);
-         }
-         free(res->res_fs.include_items);
+        while (--num >= 0) {   
+           free_incexe(res->res_fs.include_items[num]);
+        }
+        free(res->res_fs.include_items);
       }
       res->res_fs.num_includes = 0;
       if ((num=res->res_fs.num_excludes)) {
-         while (--num >= 0) {   
-            free_incexe(res->res_fs.exclude_items[num]);
-         }
-         free(res->res_fs.exclude_items);
+        while (--num >= 0) {   
+           free_incexe(res->res_fs.exclude_items[num]);
+        }
+        free(res->res_fs.exclude_items);
       }
       res->res_fs.num_excludes = 0;
       break;
    case R_POOL:
       if (res->res_pool.pool_type) {
-         free(res->res_pool.pool_type);
+        free(res->res_pool.pool_type);
       }
       if (res->res_pool.label_format) {
-         free(res->res_pool.label_format);
+        free(res->res_pool.label_format);
       }
       if (res->res_pool.cleaning_prefix) {
-         free(res->res_pool.cleaning_prefix);
+        free(res->res_pool.cleaning_prefix);
       }
       break;
    case R_SCHEDULE:
       if (res->res_sch.run) {
-         RUN *nrun, *next;
-         nrun = res->res_sch.run;
-         while (nrun) {
-            next = nrun->next;
-            free(nrun);
-            nrun = next;
-         }
+        RUN *nrun, *next;
+        nrun = res->res_sch.run;
+        while (nrun) {
+           next = nrun->next;
+           free(nrun);
+           nrun = next;
+        }
       }
       break;
    case R_JOB:
+   case R_JOBDEFS:
       if (res->res_job.RestoreWhere) {
-         free(res->res_job.RestoreWhere);
+        free(res->res_job.RestoreWhere);
       }
       if (res->res_job.RestoreBootstrap) {
-         free(res->res_job.RestoreBootstrap);
+        free(res->res_job.RestoreBootstrap);
       }
       if (res->res_job.WriteBootstrap) {
-         free(res->res_job.WriteBootstrap);
+        free(res->res_job.WriteBootstrap);
       }
       if (res->res_job.RunBeforeJob) {
-         free(res->res_job.RunBeforeJob);
+        free(res->res_job.RunBeforeJob);
       }
       if (res->res_job.RunAfterJob) {
-         free(res->res_job.RunAfterJob);
+        free(res->res_job.RunAfterJob);
+      }
+      if (res->res_job.RunAfterFailedJob) {
+        free(res->res_job.RunAfterFailedJob);
+      }
+      if (res->res_job.ClientRunBeforeJob) {
+        free(res->res_job.ClientRunBeforeJob);
+      }
+      if (res->res_job.ClientRunAfterJob) {
+        free(res->res_job.ClientRunAfterJob);
       }
       break;
    case R_MSGS:
-      if (res->res_msgs.mail_cmd)
-         free(res->res_msgs.mail_cmd);
-      if (res->res_msgs.operator_cmd)
-         free(res->res_msgs.operator_cmd);
+      if (res->res_msgs.mail_cmd) {
+        free(res->res_msgs.mail_cmd);
+      }
+      if (res->res_msgs.operator_cmd) {
+        free(res->res_msgs.operator_cmd);
+      }
       free_msgs_res((MSGS *)res);  /* free message resource */
       res = NULL;
       break;
-   case R_GROUP:
-      break;
    default:
       printf("Unknown resource type %d in free_resource.\n", type);
    }
@@ -842,9 +920,8 @@ void free_resource(int type)
    if (res) {
       free(res);
    }
-   resources[rindex].res_head = nres;
    if (nres) {
-      free_resource(type);
+      free_resource(nres, type);
    }
 }
 
@@ -854,30 +931,34 @@ void free_resource(int type)
  * pointers because they may not have been defined until 
  * later in pass 1.
  */
-void save_resource(int type, struct res_items *items, int pass)
+void save_resource(int type, RES_ITEM *items, int pass)
 {
    URES *res;
    int rindex = type - r_first;
    int i, size;
    int error = 0;
    
-   /* 
-    * Ensure that all required items are present
-    */
-   for (i=0; items[i].name; i++) {
-      if (items[i].flags & ITEM_REQUIRED) {
-            if (!bit_is_set(i, res_all.res_dir.hdr.item_present)) {  
-               Emsg2(M_ERROR_TERM, 0, "%s item is required in %s resource, but not found.\n",
-                 items[i].name, resources[rindex]);
-             }
-      }
-      /* If this triggers, take a look at lib/parse_conf.h */
-      if (i >= MAX_RES_ITEMS) {
-         Emsg1(M_ERROR_TERM, 0, "Too many items in %s resource\n", resources[rindex]);
+   /* Check Job requirements after applying JobDefs */
+   if (type != R_JOB && type != R_JOBDEFS) {
+      /* 
+       * Ensure that all required items are present
+       */
+      for (i=0; items[i].name; i++) {
+        if (items[i].flags & ITEM_REQUIRED) {
+              if (!bit_is_set(i, res_all.res_dir.hdr.item_present)) {  
+                  Emsg2(M_ERROR_TERM, 0, "%s item is required in %s resource, but not found.\n",
+                   items[i].name, resources[rindex]);
+               }
+        }
+        /* If this triggers, take a look at lib/parse_conf.h */
+        if (i >= MAX_RES_ITEMS) {
+            Emsg1(M_ERROR_TERM, 0, "Too many items in %s resource\n", resources[rindex]);
+        }
       }
    }
 
-   /* During pass 2 in each "store" routine, we looked up pointers 
+   /*
+    * During pass 2 in each "store" routine, we looked up pointers 
     * to all the resources referrenced in the current resource, now we
     * must copy their addresses from the static record to the allocated
     * record.
@@ -888,92 +969,84 @@ void save_resource(int type, struct res_items *items, int pass)
       case R_CONSOLE:
       case R_CATALOG:
       case R_STORAGE:
-      case R_GROUP:
       case R_POOL:
       case R_MSGS:
       case R_FILESET:
-         break;
+        break;
 
       /* Resources containing another resource */
       case R_DIRECTOR:
-         if ((res = (URES *)GetResWithName(R_DIRECTOR, res_all.res_dir.hdr.name)) == NULL) {
+        if ((res = (URES *)GetResWithName(R_DIRECTOR, res_all.res_dir.hdr.name)) == NULL) {
             Emsg1(M_ERROR_TERM, 0, "Cannot find Director resource %s\n", res_all.res_dir.hdr.name);
-         }
-         res->res_dir.messages = res_all.res_dir.messages;
-         break;
+        }
+        res->res_dir.messages = res_all.res_dir.messages;
+        break;
       case R_JOB:
-         if ((res = (URES *)GetResWithName(R_JOB, res_all.res_dir.hdr.name)) == NULL) {
-            Emsg1(M_ERROR_TERM, 0, "Cannot find Job resource %s\n", res_all.res_dir.hdr.name);
-         }
-         res->res_job.messages = res_all.res_job.messages;
-         res->res_job.schedule = res_all.res_job.schedule;
-         res->res_job.client   = res_all.res_job.client;
-         res->res_job.fileset  = res_all.res_job.fileset;
-         res->res_job.storage  = res_all.res_job.storage;
-         res->res_job.pool     = res_all.res_job.pool;
-         if (res->res_job.JobType == 0) {
-            Emsg1(M_ERROR_TERM, 0, "Job Type not defined for Job resource %s\n", res_all.res_dir.hdr.name);
-         }
-         if (res->res_job.level != 0) {
-            int i;
-            for (i=0; joblevels[i].level_name; i++) {
-               if (joblevels[i].level == res->res_job.level &&
-                   joblevels[i].job_type == res->res_job.JobType) {
-                  i = 0;
-                  break;
-               }
-            }
-            if (i != 0) {
-               Emsg1(M_ERROR_TERM, 0, "Inappropriate level specified in Job resource %s\n", 
-                  res_all.res_dir.hdr.name);
-            }
-         }
-         break;
+      case R_JOBDEFS:
+        if ((res = (URES *)GetResWithName(type, res_all.res_dir.hdr.name)) == NULL) {
+            Emsg1(M_ERROR_TERM, 0, "Cannot find Job resource %s\n", 
+                 res_all.res_dir.hdr.name);
+        }
+        res->res_job.messages   = res_all.res_job.messages;
+        res->res_job.schedule   = res_all.res_job.schedule;
+        res->res_job.client     = res_all.res_job.client;
+        res->res_job.fileset    = res_all.res_job.fileset;
+        res->res_job.storage    = res_all.res_job.storage;
+        res->res_job.pool       = res_all.res_job.pool;
+        res->res_job.full_pool  = res_all.res_job.full_pool;
+        res->res_job.inc_pool   = res_all.res_job.inc_pool;
+        res->res_job.dif_pool   = res_all.res_job.dif_pool;
+        res->res_job.verify_job = res_all.res_job.verify_job;
+        res->res_job.jobdefs    = res_all.res_job.jobdefs;
+        break;
       case R_COUNTER:
-         if ((res = (URES *)GetResWithName(R_COUNTER, res_all.res_counter.hdr.name)) == NULL) {
+        if ((res = (URES *)GetResWithName(R_COUNTER, res_all.res_counter.hdr.name)) == NULL) {
             Emsg1(M_ERROR_TERM, 0, "Cannot find Counter resource %s\n", res_all.res_counter.hdr.name);
-         }
-         res->res_counter.Catalog = res_all.res_counter.Catalog;
-         res->res_counter.WrapCounter = res_all.res_counter.WrapCounter;
-         break;
+        }
+        res->res_counter.Catalog = res_all.res_counter.Catalog;
+        res->res_counter.WrapCounter = res_all.res_counter.WrapCounter;
+        break;
 
       case R_CLIENT:
-         if ((res = (URES *)GetResWithName(R_CLIENT, res_all.res_client.hdr.name)) == NULL) {
+        if ((res = (URES *)GetResWithName(R_CLIENT, res_all.res_client.hdr.name)) == NULL) {
             Emsg1(M_ERROR_TERM, 0, "Cannot find Client resource %s\n", res_all.res_client.hdr.name);
-         }
-         res->res_client.catalog = res_all.res_client.catalog;
-         break;
+        }
+        res->res_client.catalog = res_all.res_client.catalog;
+        break;
       case R_SCHEDULE:
-         /* Schedule is a bit different in that it contains a RUN record
+        /*
+         * Schedule is a bit different in that it contains a RUN record
           * chain which isn't a "named" resource. This chain was linked
-          * in by run_conf.c during pass 2, so here we jam the pointer 
-          * into the Schedule resource.                         
-          */
-         if ((res = (URES *)GetResWithName(R_SCHEDULE, res_all.res_client.hdr.name)) == NULL) {
+         * in by run_conf.c during pass 2, so here we jam the pointer 
+         * into the Schedule resource.                         
+         */
+        if ((res = (URES *)GetResWithName(R_SCHEDULE, res_all.res_client.hdr.name)) == NULL) {
             Emsg1(M_ERROR_TERM, 0, "Cannot find Schedule resource %s\n", res_all.res_client.hdr.name);
-         }
-         res->res_sch.run = res_all.res_sch.run;
-         break;
+        }
+        res->res_sch.run = res_all.res_sch.run;
+        break;
       default:
          Emsg1(M_ERROR, 0, "Unknown resource type %d in save_resource.\n", type);
-         error = 1;
-         break;
+        error = 1;
+        break;
       }
       /* Note, the resource name was already saved during pass 1,
        * so here, we can just release it.
        */
       if (res_all.res_dir.hdr.name) {
-         free(res_all.res_dir.hdr.name);
-         res_all.res_dir.hdr.name = NULL;
+        free(res_all.res_dir.hdr.name);
+        res_all.res_dir.hdr.name = NULL;
       }
       if (res_all.res_dir.hdr.desc) {
-         free(res_all.res_dir.hdr.desc);
-         res_all.res_dir.hdr.desc = NULL;
+        free(res_all.res_dir.hdr.desc);
+        res_all.res_dir.hdr.desc = NULL;
       }
       return;
    }
 
-   /* The following code is only executed for pass 1 */
+   /*
+    * The following code is only executed during pass 1   
+    */
    switch (type) {
    case R_DIRECTOR:
       size = sizeof(DIRRES);
@@ -991,6 +1064,7 @@ void save_resource(int type, struct res_items *items, int pass)
       size = sizeof(CAT);
       break;
    case R_JOB:
+   case R_JOBDEFS:
       size = sizeof(JOB);
       break;
    case R_FILESET:
@@ -999,9 +1073,6 @@ void save_resource(int type, struct res_items *items, int pass)
    case R_SCHEDULE:
       size = sizeof(SCHED);
       break;
-   case R_GROUP:
-      size = sizeof(GROUP);
-      break;
    case R_POOL:
       size = sizeof(POOL);
       break;
@@ -1019,20 +1090,28 @@ void save_resource(int type, struct res_items *items, int pass)
    }
    /* Common */
    if (!error) {
+      if (type == R_JOBDEFS) {
+         Dmsg0(200, "Storing JobDefs definition.\n");
+      }
       res = (URES *)malloc(size);
       memcpy(res, &res_all, size);
       if (!resources[rindex].res_head) {
-         resources[rindex].res_head = (RES *)res; /* store first entry */
+        resources[rindex].res_head = (RES *)res; /* store first entry */
          Dmsg3(200, "Inserting first %s res: %s index=%d\n", res_to_str(type),
-               res->res_dir.hdr.name, rindex);
+              res->res_dir.hdr.name, rindex);
       } else {
-         RES *next;
-         /* Add new res to end of chain */
-         for (next=resources[rindex].res_head; next->next; next=next->next)
-            { }
-         next->next = (RES *)res;
-         Dmsg3(200, "Inserting %s res: %s index=%d\n", res_to_str(type),
-               res->res_dir.hdr.name, rindex);
+        RES *next;
+        /* Add new res to end of chain */
+        for (next=resources[rindex].res_head; next->next; next=next->next) {
+           if (strcmp(next->name, res->res_dir.hdr.name) == 0) {
+              Emsg2(M_ERROR_TERM, 0,
+                  _("Attempt to define second %s resource named \"%s\" is not permitted.\n"),
+                 resources[rindex].name, res->res_dir.hdr.name);
+           }
+        }
+        next->next = (RES *)res;
+         Dmsg4(200, "Inserting %s res: %s index=%d pass=%d\n", res_to_str(type),
+              res->res_dir.hdr.name, rindex, pass);
       }
    }
 }
@@ -1041,7 +1120,7 @@ void save_resource(int type, struct res_items *items, int pass)
  * Store JobType (backup, verify, restore)
  *
  */
-static void store_jobtype(LEX *lc, struct res_items *item, int index, int pass)
+void store_jobtype(LEX *lc, RES_ITEM *item, int index, int pass)
 {
    int token, i;   
 
@@ -1049,9 +1128,9 @@ static void store_jobtype(LEX *lc, struct res_items *item, int index, int pass)
    /* Store the type both pass 1 and pass 2 */
    for (i=0; jobtypes[i].type_name; i++) {
       if (strcasecmp(lc->str, jobtypes[i].type_name) == 0) {
-         ((JOB *)(item->value))->JobType = jobtypes[i].job_type;
-         i = 0;
-         break;
+        *(int *)(item->value) = jobtypes[i].job_type;
+        i = 0;
+        break;
       }
    }
    if (i != 0) {
@@ -1065,7 +1144,7 @@ static void store_jobtype(LEX *lc, struct res_items *item, int index, int pass)
  * Store Job Level (Full, Incremental, ...)
  *
  */
-static void store_level(LEX *lc, struct res_items *item, int index, int pass)
+void store_level(LEX *lc, RES_ITEM *item, int index, int pass)
 {
    int token, i;
 
@@ -1073,9 +1152,9 @@ static void store_level(LEX *lc, struct res_items *item, int index, int pass)
    /* Store the level pass 2 so that type is defined */
    for (i=0; joblevels[i].level_name; i++) {
       if (strcasecmp(lc->str, joblevels[i].level_name) == 0) {
-         ((JOB *)(item->value))->level = joblevels[i].level;
-         i = 0;
-         break;
+        *(int *)(item->value) = joblevels[i].level;
+        i = 0;
+        break;
       }
    }
    if (i != 0) {
@@ -1085,16 +1164,16 @@ static void store_level(LEX *lc, struct res_items *item, int index, int pass)
    set_bit(index, res_all.hdr.item_present);
 }
 
-static void store_replace(LEX *lc, struct res_items *item, int index, int pass)
+void store_replace(LEX *lc, RES_ITEM *item, int index, int pass)
 {
    int token, i;
    token = lex_get_token(lc, T_NAME);
    /* Scan Replacement options */
    for (i=0; ReplaceOptions[i].name; i++) {
       if (strcasecmp(lc->str, ReplaceOptions[i].name) == 0) {
-         *(int *)(item->value) = ReplaceOptions[i].token;
-         i = 0;
-         break;
+        *(int *)(item->value) = ReplaceOptions[i].token;
+        i = 0;
+        break;
       }
    }
    if (i != 0) {
@@ -1104,6 +1183,35 @@ static void store_replace(LEX *lc, struct res_items *item, int index, int pass)
    set_bit(index, res_all.hdr.item_present);
 }
 
+/* 
+ * Store ACL (access control list)
+ *
+ */
+void store_acl(LEX *lc, RES_ITEM *item, int index, int pass)
+{
+   int token;
+
+   for (;;) {
+      token = lex_get_token(lc, T_NAME);
+      if (pass == 1) {
+        if (((alist **)item->value)[item->code] == NULL) {   
+           ((alist **)item->value)[item->code] = new alist(10, owned_by_alist);
+//          Dmsg1(400, "Defined new ACL alist at %d\n", item->code);
+        }
+        ((alist **)item->value)[item->code]->append(bstrdup(lc->str));
+//       Dmsg2(400, "Appended to %d %s\n", item->code, lc->str);
+      }
+      token = lex_get_token(lc, T_ALL);
+      if (token == T_COMMA) {
+        continue;                    /* get another ACL */
+      }
+      break;
+   }
+   set_bit(index, res_all.hdr.item_present);
+}
+
+
+#ifdef old_deprecated_code
 /* 
  * Store backup/verify info for Job record 
  *
@@ -1111,7 +1219,7 @@ static void store_replace(LEX *lc, struct res_items *item, int index, int pass)
  *
  *    Backup = Client=<client-name> FileSet=<FileSet-name> Level=<level>
  */
-static void store_backup(LEX *lc, struct res_items *item, int index, int pass)
+static void store_backup(LEX *lc, RES_ITEM *item, int index, int pass)
 {
    int token, i;
    RES *res;
@@ -1122,7 +1230,7 @@ static void store_backup(LEX *lc, struct res_items *item, int index, int pass)
    
    ((JOB *)(item->value))->JobType = item->code;
    while ((token = lex_get_token(lc, T_ALL)) != T_EOL) {
-      int found;
+      bool found = false;
 
       Dmsg1(150, "store_backup got token=%s\n", lex_tok_to_str(token));
       
@@ -1130,61 +1238,60 @@ static void store_backup(LEX *lc, struct res_items *item, int index, int pass)
          scan_err1(lc, "Expected a backup/verify keyword, got: %s", lc->str);
       }
       Dmsg1(190, "Got keyword: %s\n", lc->str);
-      found = FALSE;
       for (i=0; BakVerFields[i].name; i++) {
-         if (strcasecmp(lc->str, BakVerFields[i].name) == 0) {
-            found = TRUE;
-            if (lex_get_token(lc, T_ALL) != T_EQUALS) {
+        if (strcasecmp(lc->str, BakVerFields[i].name) == 0) {
+           found = true;
+           if (lex_get_token(lc, T_ALL) != T_EQUALS) {
                scan_err1(lc, "Expected an equals, got: %s", lc->str);
-            }
-            token = lex_get_token(lc, T_NAME);
+           }
+           token = lex_get_token(lc, T_NAME);
             Dmsg1(190, "Got value: %s\n", lc->str);
-            switch (BakVerFields[i].token) {
-               case 'C':
-                  /* Find Client Resource */
-                  if (pass == 2) {
-                     res = GetResWithName(R_CLIENT, lc->str);
-                     if (res == NULL) {
-                        scan_err1(lc, "Could not find specified Client Resource: %s",
-                                   lc->str);
-                     }
-                     res_all.res_job.client = (CLIENT *)res;
-                  }
-                  break;
-               case 'F':
-                  /* Find FileSet Resource */
-                  if (pass == 2) {
-                     res = GetResWithName(R_FILESET, lc->str);
-                     if (res == NULL) {
-                        scan_err1(lc, "Could not find specified FileSet Resource: %s\n",
-                                    lc->str);
-                     }
-                     res_all.res_job.fileset = (FILESET *)res;
-                  }
-                  break;
-               case 'L':
-                  /* Get level */
-                  for (i=0; joblevels[i].level_name; i++) {
-                     if (joblevels[i].job_type == item->code && 
-                          strcasecmp(lc->str, joblevels[i].level_name) == 0) {
-                        ((JOB *)(item->value))->level = joblevels[i].level;
-                        i = 0;
-                        break;
-                     }
-                  }
-                  if (i != 0) {
-                     scan_err1(lc, "Expected a Job Level keyword, got: %s", lc->str);
-                  }
-                  break;
-            } /* end switch */
-            break;
-         } /* end if strcmp() */
+           switch (BakVerFields[i].token) {
+            case 'C':
+              /* Find Client Resource */
+              if (pass == 2) {
+                 res = GetResWithName(R_CLIENT, lc->str);
+                 if (res == NULL) {
+                     scan_err1(lc, "Could not find specified Client Resource: %s",
+                               lc->str);
+                 }
+                 res_all.res_job.client = (CLIENT *)res;
+              }
+              break;
+            case 'F':
+              /* Find FileSet Resource */
+              if (pass == 2) {
+                 res = GetResWithName(R_FILESET, lc->str);
+                 if (res == NULL) {
+                     scan_err1(lc, "Could not find specified FileSet Resource: %s\n",
+                                lc->str);
+                 }
+                 res_all.res_job.fileset = (FILESET *)res;
+              }
+              break;
+            case 'L':
+              /* Get level */
+              for (i=0; joblevels[i].level_name; i++) {
+                 if (joblevels[i].job_type == item->code && 
+                      strcasecmp(lc->str, joblevels[i].level_name) == 0) {
+                    ((JOB *)(item->value))->level = joblevels[i].level;
+                    i = 0;
+                    break;
+                 }
+              }
+              if (i != 0) {
+                  scan_err1(lc, "Expected a Job Level keyword, got: %s", lc->str);
+              }
+              break;
+           } /* end switch */
+           break;
+        } /* end if strcmp() */
       } /* end for */
       if (!found) {
          scan_err1(lc, "%s not a valid Backup/verify keyword", lc->str);
       }
    } /* end while */
-   lc->options = options;             /* reset original options */
+   lc->options = options;            /* reset original options */
    set_bit(index, res_all.hdr.item_present);
 }
 
@@ -1194,7 +1301,7 @@ static void store_backup(LEX *lc, struct res_items *item, int index, int pass)
  *    Restore = JobId=<job-id> Where=<root-directory> Replace=<options> Bootstrap=<file>
  *
  */
-static void store_restore(LEX *lc, struct res_items *item, int index, int pass)
+static void store_restore(LEX *lc, RES_ITEM *item, int index, int pass)
 {
    int token, i;
    RES *res;
@@ -1206,99 +1313,99 @@ static void store_restore(LEX *lc, struct res_items *item, int index, int pass)
    
    ((JOB *)(item->value))->JobType = item->code;
    while ((token = lex_get_token(lc, T_ALL)) != T_EOL) {
-      int found; 
+      bool found = false;
 
       if (token != T_IDENTIFIER && token != T_UNQUOTED_STRING && token != T_QUOTED_STRING) {
          scan_err1(lc, "expected a name, got: %s", lc->str);
       }
-      found = FALSE;
       for (i=0; RestoreFields[i].name; i++) {
          Dmsg1(190, "Restore kw=%s\n", lc->str);
-         if (strcasecmp(lc->str, RestoreFields[i].name) == 0) {
-            found = TRUE;
-            if (lex_get_token(lc, T_ALL) != T_EQUALS) {
+        if (strcasecmp(lc->str, RestoreFields[i].name) == 0) {
+           found = true;
+           if (lex_get_token(lc, T_ALL) != T_EQUALS) {
                scan_err1(lc, "Expected an equals, got: %s", lc->str);
-            }
-            token = lex_get_token(lc, T_ALL);
+           }
+           token = lex_get_token(lc, T_ALL);
             Dmsg1(190, "Restore value=%s\n", lc->str);
-            switch (RestoreFields[i].token) {
-               case 'B':
-                  /* Bootstrap */
-                  if (token != T_IDENTIFIER && token != T_UNQUOTED_STRING && token != T_QUOTED_STRING) {
-                     scan_err1(lc, "Expected a Restore bootstrap file, got: %s", lc->str);
-                  }
-                  if (pass == 1) {
-                     res_all.res_job.RestoreBootstrap = bstrdup(lc->str);
-                  }
-                  break;
-               case 'C':
-                  /* Find Client Resource */
-                  if (pass == 2) {
-                     res = GetResWithName(R_CLIENT, lc->str);
-                     if (res == NULL) {
-                        scan_err1(lc, "Could not find specified Client Resource: %s",
-                                   lc->str);
-                     }
-                     res_all.res_job.client = (CLIENT *)res;
-                  }
-                  break;
-               case 'F':
-                  /* Find FileSet Resource */
-                  if (pass == 2) {
-                     res = GetResWithName(R_FILESET, lc->str);
-                     if (res == NULL) {
-                        scan_err1(lc, "Could not find specified FileSet Resource: %s\n",
-                                    lc->str);
-                     }
-                     res_all.res_job.fileset = (FILESET *)res;
-                  }
-                  break;
-               case 'J':
-                  /* JobId */
-                  if (token != T_NUMBER) {
-                     scan_err1(lc, "expected an integer number, got: %s", lc->str);
-                  }
-                  errno = 0;
-                  res_all.res_job.RestoreJobId = strtol(lc->str, NULL, 0);
-                  Dmsg1(190, "RestorJobId=%d\n", res_all.res_job.RestoreJobId);
-                  if (errno != 0) {
-                     scan_err1(lc, "expected an integer number, got: %s", lc->str);
-                  }
-                  break;
-               case 'W':
-                  /* Where */
-                  if (token != T_IDENTIFIER && token != T_UNQUOTED_STRING && token != T_QUOTED_STRING) {
-                     scan_err1(lc, "Expected a Restore root directory, got: %s", lc->str);
-                  }
-                  if (pass == 1) {
-                     res_all.res_job.RestoreWhere = bstrdup(lc->str);
-                  }
-                  break;
-               case 'R':
-                  /* Replacement options */
-                  if (token != T_IDENTIFIER && token != T_UNQUOTED_STRING && token != T_QUOTED_STRING) {
-                     scan_err1(lc, "Expected a keyword name, got: %s", lc->str);
-                  }
-                  /* Fix to scan Replacement options */
-                  for (i=0; ReplaceOptions[i].name; i++) {
-                     if (strcasecmp(lc->str, ReplaceOptions[i].name) == 0) {
-                         ((JOB *)(item->value))->replace = ReplaceOptions[i].token;
-                        i = 0;
-                        break;
-                     }
-                  }
-                  if (i != 0) {
-                     scan_err1(lc, "Expected a Restore replacement option, got: %s", lc->str);
-                  }
-                  break;
-            } /* end switch */
-            break;
-         } /* end if strcmp() */
+           switch (RestoreFields[i].token) {
+            case 'B':
+              /* Bootstrap */
+              if (token != T_IDENTIFIER && token != T_UNQUOTED_STRING && token != T_QUOTED_STRING) {
+                  scan_err1(lc, "Expected a Restore bootstrap file, got: %s", lc->str);
+              }
+              if (pass == 1) {
+                 res_all.res_job.RestoreBootstrap = bstrdup(lc->str);
+              }
+              break;
+            case 'C':
+              /* Find Client Resource */
+              if (pass == 2) {
+                 res = GetResWithName(R_CLIENT, lc->str);
+                 if (res == NULL) {
+                     scan_err1(lc, "Could not find specified Client Resource: %s",
+                               lc->str);
+                 }
+                 res_all.res_job.client = (CLIENT *)res;
+              }
+              break;
+            case 'F':
+              /* Find FileSet Resource */
+              if (pass == 2) {
+                 res = GetResWithName(R_FILESET, lc->str);
+                 if (res == NULL) {
+                     scan_err1(lc, "Could not find specified FileSet Resource: %s\n",
+                                lc->str);
+                 }
+                 res_all.res_job.fileset = (FILESET *)res;
+              }
+              break;
+            case 'J':
+              /* JobId */
+              if (token != T_NUMBER) {
+                  scan_err1(lc, "expected an integer number, got: %s", lc->str);
+              }
+              errno = 0;
+              res_all.res_job.RestoreJobId = strtol(lc->str, NULL, 0);
+               Dmsg1(190, "RestorJobId=%d\n", res_all.res_job.RestoreJobId);
+              if (errno != 0) {
+                  scan_err1(lc, "expected an integer number, got: %s", lc->str);
+              }
+              break;
+            case 'W':
+              /* Where */
+              if (token != T_IDENTIFIER && token != T_UNQUOTED_STRING && token != T_QUOTED_STRING) {
+                  scan_err1(lc, "Expected a Restore root directory, got: %s", lc->str);
+              }
+              if (pass == 1) {
+                 res_all.res_job.RestoreWhere = bstrdup(lc->str);
+              }
+              break;
+            case 'R':
+              /* Replacement options */
+              if (token != T_IDENTIFIER && token != T_UNQUOTED_STRING && token != T_QUOTED_STRING) {
+                  scan_err1(lc, "Expected a keyword name, got: %s", lc->str);
+              }
+              /* Fix to scan Replacement options */
+              for (i=0; ReplaceOptions[i].name; i++) {
+                 if (strcasecmp(lc->str, ReplaceOptions[i].name) == 0) {
+                     ((JOB *)(item->value))->replace = ReplaceOptions[i].token;
+                    i = 0;
+                    break;
+                 }
+              }
+              if (i != 0) {
+                  scan_err1(lc, "Expected a Restore replacement option, got: %s", lc->str);
+              }
+              break;
+           } /* end switch */
+           break;
+        } /* end if strcmp() */
       } /* end for */
       if (!found) {
          scan_err1(lc, "%s not a valid Restore keyword", lc->str);
       }
    } /* end while */
-   lc->options = options;             /* reset original options */
+   lc->options = options;            /* reset original options */
    set_bit(index, res_all.hdr.item_present);
 }
+#endif