diff --git a/src/borg/archiver.py b/src/borg/archiver.py index 20d387d0c..8eb8bb28b 100644 --- a/src/borg/archiver.py +++ b/src/borg/archiver.py @@ -1744,16 +1744,13 @@ class Archiver: def repo_validate(section, name, value=None, check_value=True): if section not in ['repository', ]: raise ValueError('Invalid section') - if name in ['segments_per_dir', 'max_segment_size', 'last_segment_checked', ]: + if name in ['segments_per_dir', 'last_segment_checked', ]: if check_value: try: int(value) except ValueError: raise ValueError('Invalid value') from None - if name == 'max_segment_size': - if int(value) >= MAX_SEGMENT_SIZE_LIMIT: - raise ValueError('Invalid value: max_segment_size >= %d' % MAX_SEGMENT_SIZE_LIMIT) - elif name in ['additional_free_space', 'storage_quota', ]: + elif name in ['max_segment_size', 'additional_free_space', 'storage_quota', ]: if check_value: try: parse_file_size(value) @@ -1762,6 +1759,9 @@ class Archiver: if name == 'storage_quota': if parse_file_size(value) < parse_file_size('10M'): raise ValueError('Invalid value: storage_quota < 10M') + elif name == 'max_segment_size': + if parse_file_size(value) >= MAX_SEGMENT_SIZE_LIMIT: + raise ValueError('Invalid value: max_segment_size >= %d' % MAX_SEGMENT_SIZE_LIMIT) elif name in ['append_only', ]: if check_value and value not in ['0', '1']: raise ValueError('Invalid value') diff --git a/src/borg/repository.py b/src/borg/repository.py index 223b1e2ba..c0926b58a 100644 --- a/src/borg/repository.py +++ b/src/borg/repository.py @@ -432,7 +432,7 @@ class Repository: if 'repository' not in self.config.sections() or self.config.getint('repository', 'version') != 1: self.close() raise self.InvalidRepository(path) - self.max_segment_size = self.config.getint('repository', 'max_segment_size') + self.max_segment_size = parse_file_size(self.config.get('repository', 'max_segment_size')) if self.max_segment_size >= MAX_SEGMENT_SIZE_LIMIT: self.close() raise self.InvalidRepositoryConfig(path, 'max_segment_size >= %d' % MAX_SEGMENT_SIZE_LIMIT) # issue 3592