Menu

#36 Extend Corrupted File Detection

v2.0.1
closed-fixed
Federico
None
5
2016-12-29
2016-11-29
theseus
No

Hi there,

we had some bug report due a doc file which was corrupted, but doesn't result in a CFCorruptedFileException exception.
Due the not checked sector index the library crashes later with a OutOfRange while accessing the sector table. I have some bugfix including a unit test and test document for that.
It would be great if you can take a look and merge the changes into your official repo:
https://github.com/fxeu/OpenMCDF/pull/1

By the way: It is rather complex for you and others to do a collaborate work on your project if using svn and/or sourceforge. Have you thought about moving to some platform like github or gitlabs? There are already multiple forks on that platforms and it would be great to merge them all together back into one official repo.

Thanks and I am looking forward to hearing from you.

Marco

Discussion

  • theseus

    theseus - 2016-12-01

    I created the creates based on your trunk, which was wrong. I recreated them on your dev here: https://github.com/fxeu/OpenMCDF/pull/2

     
  • Federico

    Federico - 2016-12-15
    • status: open --> accepted
    • assigned_to: Federico
     
  • Federico

    Federico - 2016-12-15

    Closed in dev.

    Thank you very much for precise reporting and unit test: I have integrated your code in dev branch.
    And Yes, I think that in a near future OpenMcdf will migrate to github.

    Best Regards,
    Federico

     
  • Federico

    Federico - 2016-12-29
    • status: accepted --> closed-fixed
     
  • Federico

    Federico - 2016-12-29

    Closed in github migration (2.1)

     

Log in to post a comment.

MongoDB Logo MongoDB