这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@weidenba
Copy link
Contributor

@weidenba weidenba commented Nov 2, 2018

Extraction plugin for dahua firmware images.

Dahua uses zip containers but change the magic bytes from 'PK' to 'DH'.

@codecov-io
Copy link

codecov-io commented Nov 2, 2018

Codecov Report

Merging #172 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
+ Coverage   95.37%   95.37%   +<.01%     
==========================================
  Files         314      316       +2     
  Lines       15704    15731      +27     
==========================================
+ Hits        14978    15004      +26     
- Misses        726      727       +1
Impacted Files Coverage Δ
src/plugins/unpacking/dahua/test/test_dahua.py 100% <100%> (ø)
src/plugins/unpacking/dahua/code/dahua.py 100% <100%> (ø)
...test/integration/scheduler/test_cycle_with_tags.py 98.11% <0%> (-1.89%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 375d546...f637c9f. Read the comment docs.

@weidenba weidenba self-assigned this Nov 2, 2018


def unpack_function(file_path, tmp_dir):
output_file_path = Path(tmp_dir, 'dahua_firmware.zip')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

imho it would look better in the file tree (and be more intuitive) if the original file name was kept and a ".zip" was appended

@weidenba weidenba merged commit 2149ea5 into master Nov 5, 2018
@weidenba weidenba deleted the dahua_unpack_plugin branch November 5, 2018 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants