verdammelt / tnef

tnef
GNU General Public License v2.0
58 stars 21 forks source link

Error while parsing file with MAPI multi-value fields. #3

Closed verdammelt closed 10 years ago

verdammelt commented 10 years ago

Certain winmail.dat files contain MAPI fields which are 'multi-value' fields. These are inconsistently handled by tnef and case result in a termination of processing with the error message:

!!MULTI_VALUE_FLAG seen (0x1205 0x02) Assertion failed: (!mvf), function mapi_attr_read, file mapi_attr.c, line 250. Abort trap: 6

verdammelt commented 10 years ago

See pull request #6

dtimms commented 10 years ago

Just a note that attempting: make check is showing a test failure in:

FAIL: multi-value-attribute.test
Testsuite summary for tnef 1.4.11
============================================================================
# TOTAL: 13
# PASS:  12
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0
============================================================================
See tests/files/test-suite.log

Given this issue relates to improvements for mult-value fields, this might be related to these patches.

https://kojipkgs.fedoraproject.org//work/tasks/11/7260011/build.log I couldn't see a tests/files/test-suite.log file.

verdammelt commented 10 years ago

I don't have access to a fedora system and it doesn't happen on my system.

Can you send me the files tests/files/multi-value-attribute.* from after the build.

Thanks.

ileGITimo commented 10 years ago

I've tried this on a Fedora 20 system. Error seems to be in the log for multi-value-attribute: ./datafiles/multi-value-attribute.tnef does not exist. Test Failed! Seems like the multi-valute-attribute.tnef is missing

dtimms commented 10 years ago

Hi, I must have looked in the wrong folder for the file noted in the build log. It is present. I've archived the build result related to tests and multivalue-... I included files each side of them. It's a gzip; couldn't see how to attach that, so uploaded to: http://members.iinet.net.au/~timmsy/tnef/tnef-1.4.11-tests.fedora20_build.tar.bz2

verdammelt commented 10 years ago

I'm going to open a new issue for this fedora issue and then see about getting a fedora image and attempt to replicate.