Bug 196714 - off by one error in 7zip code?
Summary: off by one error in 7zip code?
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: p7zip
Version: 5
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Matthias Saou
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-06-26 17:22 UTC by Armijn Hemel
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-06-29 17:36:07 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Armijn Hemel 2006-06-26 17:22:03 UTC
Description of problem:

There might be a off by one error in the 7zip code (I think...not sure).
In the file p7zip_4.39/7zip/Archive/7z/7zHeader.cpp it says:

Byte kSignature[kSignatureSize] = {'7' + 1, 'z', 0xBC, 0xAF, 0x27, 0x1C};

but doesn't this make the header of a 7zip package start with "8" instead of "7"?

Version-Release number of selected component (if applicable):
4.39.1.fc5

How reproducible:
n/a

Steps to Reproduce:
1. n/a
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Matthias Saou 2006-06-26 17:46:18 UTC
This kind of bug report would be much better sent upstream, as the developers
know the code much better than we do, and could probably confirm in no time. I'd
really appreciate if you made the report and got back to this bug entry with a
patch if needed...

Comment 2 Matthias Saou 2006-06-29 17:36:07 UTC
Maybe a fix would be to rename the package 8zip, then? :-)
I've updated the FC devel package to 4.42. I'll close this bug for now, please
report the (potential) problem upstream so that it gets fixed there. The I'll
update the package and get the fix in that way, thanks.

Comment 3 Armijn Hemel 2006-06-29 21:54:06 UTC
Well, it does generate correct headers, so I think it is just some unclear
coding, but it eventually gets fixed. I will dig into this someday. You can
leave it closed for now :)


Note You need to log in before you can comment on or make changes to this bug.