Skip to content

Commit 04b7daa

Browse files
committed
Assign new string constant DFXML_VERSION as default value
Signed-off-by: Alex Nelson <[email protected]>
1 parent bb0f93d commit 04b7daa

18 files changed

+42
-36
lines changed

dfxml/__init__.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,9 @@
6161

6262
tsk_virtual_filenames = set(['$FAT1','$FAT2'])
6363

64+
# This version string is the version of the DFXML Schema.
65+
DFXML_VERSION = "2.0.0-beta.0"
66+
6467
XMLNS_DC = "http://purl.org/dc/elements/1.1/"
6568
XMLNS_DFXML = "http://www.forensicswiki.org/wiki/Category:Digital_Forensics_XML"
6669
XMLNS_DELTA = "http://www.forensicswiki.org/wiki/Forensic_Disk_Differencing"

dfxml/bin/Extractor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ def extract_files(image_path, outdir, dfxml_path=None, file_predicate=is_file, f
7070
_path_for_iterparse = dfxml_path or image_path
7171

7272
#Set up base manifest to track extracted files
73-
base_manifest = Objects.DFXMLObject(version="2.0.0-beta.0")
73+
base_manifest = Objects.DFXMLObject()
7474
base_manifest.program = sys.argv[0]
7575
if sys.argv[0] == os.path.basename(__file__):
7676
base_manifest.program_version = __version__

dfxml/bin/cat_fileobjects.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def main():
3939
<dfxml
4040
xmlns="%s"
4141
xmlns:delta="%s"
42-
version="2.0.0-beta.0">
42+
version="%s">
4343
<metadata/>
4444
<creator>
4545
<program>%s</program>
@@ -51,7 +51,7 @@ def main():
5151
<source>
5252
<image_filename>%s</image_filename>
5353
</source>\
54-
""" % (dfxml.XMLNS_DFXML, dfxml.XMLNS_DELTA, sys.argv[0], __version__, " ".join(sys.argv), args.filename))
54+
""" % (dfxml.XMLNS_DFXML, dfxml.XMLNS_DELTA, dfxml.DFXML_VERSION, sys.argv[0], __version__, " ".join(sys.argv), args.filename))
5555

5656
ET.register_namespace("delta", dfxml.XMLNS_DELTA)
5757

dfxml/bin/cat_partitions.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
_logger = logging.getLogger(os.path.basename(__file__))
4040

4141
def main():
42-
d = Objects.DFXMLObject(version="2.0.0-beta.0")
42+
d = Objects.DFXMLObject()
4343

4444
d.program = sys.argv[0]
4545
d.program_version = __version__

dfxml/bin/idifference.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,6 +365,7 @@ def to_xml(self):
365365
exit(1)
366366

367367
metadict = dict()
368+
metadict["DFXML_VERSION"] = dfxml.DFXML_VERSION
368369
metadict["XMLNS_DFXML"] = dfxml.XMLNS_DFXML
369370
metadict["XMLNS_DELTA"] = dfxml.XMLNS_DELTA
370371
metadict["program"] = sys.argv[0]
@@ -377,7 +378,7 @@ def to_xml(self):
377378
xmlfile.write("""\
378379
<?xml version="1.0" encoding="UTF-8"?>
379380
<dfxml
380-
version="2.0.0-beta.0"
381+
version="%(DFXML_VERSION)s"
381382
xmlns='%(XMLNS_DFXML)s'
382383
xmlns:dc='http://purl.org/dc/elements/1.1/'
383384
xmlns:delta='%(XMLNS_DELTA)s'

dfxml/bin/make_differential_dfxml.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ def make_differential_dfxml(
103103
_logger.debug("diff_mask_set = " + repr(diff_mask_set))
104104

105105
#d: The container DFXMLObject, ultimately returned.
106-
d = Objects.DFXMLObject(version="2.0.0-beta.0")
106+
d = Objects.DFXMLObject()
107107
if sys.argv[0] == os.path.basename(__file__):
108108
d.program = sys.argv[0]
109109
d.program_version = __version__

dfxml/bin/walk_to_dfxml.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ def main() -> None:
210210
using_threading = False
211211
_logger.warning("Python queue support not available. (If running Ubuntu, this is in package python3-queuelib.) Running in single thread only.")
212212

213-
dobj = Objects.DFXMLObject(version="2.0.0-beta.0")
213+
dobj = Objects.DFXMLObject()
214214
dobj.program = sys.argv[0]
215215
dobj.program_version = __version__
216216
dobj.command_line = " ".join(sys.argv)

dfxml/objects.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -232,11 +232,11 @@ def child_objects(self):
232232

233233
class DFXMLObject(AbstractParentObject):
234234

235-
def __init__(self, *args, **kwargs) -> None:
235+
def __init__(self, *args: typing.Any, version: str = dfxml.DFXML_VERSION, **kwargs: typing.Any) -> None:
236236
self.command_line = kwargs.get("command_line")
237237
self.program = kwargs.get("program")
238238
self.program_version = kwargs.get("program_version")
239-
self.version = kwargs.get("version")
239+
self.version = version
240240
self.sources = kwargs.get("sources", [])
241241
self.dc = kwargs.get("dc", dict())
242242
self.externals = kwargs.get("externals", OtherNSElementList())

dfxml/writer.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@
1616
import __main__
1717
import psutil
1818

19+
from dfxml import DFXML_VERSION
20+
1921
__version__="0.2.0"
2022

2123
###
@@ -84,7 +86,7 @@ def __init__(self,heartbeat=None,filename=None,prettyprint=False):
8486
"""
8587
self.t0 = time.time()
8688
self.tlast = time.time()
87-
self.doc = ET.Element('dfxml', {'version':'2.0.0-beta.0'})
89+
self.doc = ET.Element('dfxml', {'version': DFXML_VERSION})
8890
self.add_DFXML_creator(self.doc)
8991
self.filename = filename
9092
self.timers = {}

tests/misc_object_tests/ByteRun_test.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ def test_glomming_fill():
246246
raise
247247

248248
def test_hash_properties():
249-
dobj = Objects.DFXMLObject(version="2.0.0-beta.0")
249+
dobj = Objects.DFXMLObject()
250250

251251
fobj = Objects.FileObject()
252252
dobj.append(fobj)

0 commit comments

Comments
 (0)