Use one MAX_BLOCK_SIZE and set to 20M

diff --git a/bacula/src/stored/block.h b/bacula/src/stored/block.h
index 1e126823db58c583fe8fe76dfd11637c688139c0..6b23d3ebe38d06b2c0fa4eb0cbd00b130880275e 100644 (file)
--- a/bacula/src/stored/block.h
+++ b/bacula/src/stored/block.h
@@ -26,7 +26,7 @@
 #ifndef __BLOCK_H
 #define __BLOCK_H 1
 
-#define MAX_BLOCK_LENGTH  20000000      /* this is a sort of sanity check */
+#define MAX_BLOCK_SIZE  20000000        /* this is a sort of sanity check */
 #define DEFAULT_BLOCK_SIZE (512 * 126)  /* 64,512 N.B. do not use 65,636 here */
 #define MIN_DEDUP_BLOCK_SIZE (512 * 2)  /* Minimum block (bucket) size */
 

diff --git a/bacula/src/stored/block_util.c b/bacula/src/stored/block_util.c
index e18605345f5974d905ed61d59e5636835000766c..ed18bd8573261fdcf3b85e4d1a9f2a22a321f585 100644 (file)
--- a/bacula/src/stored/block_util.c
+++ b/bacula/src/stored/block_util.c
@@ -434,7 +434,7 @@ bool unser_block_header(DCR *dcr, DEVICE *dev, DEV_BLOCK *block)
    }
 
    /* Sanity check */
-   if (block_len > MAX_BLOCK_LENGTH) {
+   if (block_len > MAX_BLOCK_SIZE) {
       dev->dev_errno = EIO;
       Mmsg3(dev->errmsg,  _("Volume data error at %u:%u! Block length %u is insane (too large), probably due to a bad archive.\n"),
          dev->file, dev->block_num, block_len);

diff --git a/bacula/src/stored/init_dev.c b/bacula/src/stored/init_dev.c
index 01d6eb6f5f575fa6aac22f1a853c3ac0e0ac5d87..9f4999a89e9432eb4b7a7050ce87c452f53ca239 100644 (file)
--- a/bacula/src/stored/init_dev.c
+++ b/bacula/src/stored/init_dev.c
@@ -312,10 +312,10 @@ void DEVICE::device_generic_init(JCR *jcr, DEVRES *device)
       Jmsg(jcr, M_ERROR_TERM, 0, _("[SA0005] Min block size > max on device %s\n"),
            dev->print_name());
    }
-   if (dev->max_block_size > 4096000) {
+   if (dev->max_block_size > MAX_BLOCK_SIZE) {
       Jmsg3(jcr, M_ERROR, 0, _("[SA0006] Block size %u on device %s is too large, using default %u\n"),
          dev->max_block_size, dev->print_name(), DEFAULT_BLOCK_SIZE);
-      dev->max_block_size = 0;
+      dev->max_block_size = DEFAULT_BLOCK_SIZE ;
    }
    if (dev->max_block_size % TAPE_BSIZE != 0) {
       Jmsg3(jcr, M_WARNING, 0, _("[SW0007] Max block size %u not multiple of device %s block size=%d.\n"),

diff --git a/bacula/src/stored/record_read.c b/bacula/src/stored/record_read.c
index a08acbe2f75dda9cd5ae69bc0ad4cd3a74b48be2..3fd1f04432940a7df46b917b5bbcb31d087630c9 100644 (file)
--- a/bacula/src/stored/record_read.c
+++ b/bacula/src/stored/record_read.c
@@ -159,7 +159,7 @@ static bool read_header(DCR *dcr, DEV_BLOCK *block, DEV_RECORD *rec)
    }
 
    /* Sanity check */
-   if (rec->data_bytes >= MAX_BLOCK_LENGTH) {
+   if (rec->data_bytes >= MAX_BLOCK_SIZE) {
       Dmsg0(dbgep, "=== rpath 11b maxlen too big\n");
       /*
        * Something is wrong, force read of next block, abort
@@ -168,7 +168,7 @@ static bool read_header(DCR *dcr, DEV_BLOCK *block, DEV_RECORD *rec)
       rec->state_bits |= (REC_NO_HEADER | REC_BLOCK_EMPTY);
       empty_block(block);
       Jmsg2(dcr->jcr, M_WARNING, 0, _("Sanity check failed. maxlen=%d datalen=%d. Block discarded.\n"),
-         MAX_BLOCK_LENGTH, rec->data_bytes);
+         MAX_BLOCK_SIZE, rec->data_bytes);
       return false;
    }
 

diff --git a/bacula/src/stored/stored_conf.c b/bacula/src/stored/stored_conf.c
index 43a47d9a2a24747b2b9d008083dffa9eb6eb440d..19997f8c6a8e2b0e7c00c0db825b0254f948d282 100644 (file)
--- a/bacula/src/stored/stored_conf.c
+++ b/bacula/src/stored/stored_conf.c
@@ -439,15 +439,15 @@ void store_uri_style(LEX *lc, RES_ITEM *item, int index, int pass)
 
 
 /*
- * Store Maximum Block Size, and check it is not greater than MAX_BLOCK_LENGTH
+ * Store Maximum Block Size, and check it is not greater than MAX_BLOCK_SIZE
  *
  */
 void store_maxblocksize(LEX *lc, RES_ITEM *item, int index, int pass)
 {
    store_size32(lc, item, index, pass);
-   if (*(uint32_t *)(item->value) > MAX_BLOCK_LENGTH) {
+   if (*(uint32_t *)(item->value) > MAX_BLOCK_SIZE) {
       scan_err2(lc, _("Maximum Block Size configured value %u is greater than allowed maximum: %u"),
-         *(uint32_t *)(item->value), MAX_BLOCK_LENGTH );
+         *(uint32_t *)(item->value), MAX_BLOCK_SIZE );
    }
 }