Review Request 128233: Don't trust files claiming to be created on unix more than other files

[Sune Vuorela]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128233/
-----------------------------------------------------------

(Updated July 4, 2016, 6:38 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and David Faure.


Changes
-------

Submitted with commit b7d8fce0a7167f4cea7127c6142ea10c92ff546d by Sune Vuorela to branch master.


Repository: karchive


Description
-------

Don't trust files claiming to be created on unix more than other files
    
    For some historical reasons, we special case zip files claiming to be
    created on unix and trust the content regarding file rights a bit better.
    
    Zip files in the wild have shown to violate this, so don't trust them.
    
    Thanks to Jonathan Marten for the test case
    
    BUG: 364071


Diffs
-----

  autotests/data/unusual_but_valid_364071.zip PRE-CREATION 
  autotests/karchivetest.h 5a6375c 
  autotests/karchivetest.cpp dd03dac 
  src/kzip.cpp e7e8477 

Diff: https://git.reviewboard.kde.org/r/128233/diff/


Testing
-------


Thanks,

Sune Vuorela

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160704/107cc230/attachment.html>