Bug 76107 - File Corruption: RT file gets corrupt due to mismatch in the reference id's in document.xml and comment.xml
Summary: File Corruption: RT file gets corrupt due to mismatch in the reference id's i...
Status: CLOSED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.3.0.0.alpha0+ Master
Hardware: Other All
: medium normal
Assignee: Umesh Kadam
URL:
Whiteboard: target:4.3.0
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-13 09:23 UTC by Umesh Kadam
Modified: 2014-03-18 13:50 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:

Attachments
File has comments for paragraphs. (14.00 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2014-03-13 09:23 UTC, Umesh Kadam 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Umesh Kadam 2014-03-13 09:23:05 UTC 
Issue Simulation:
---------
1) open the file in LO.
2) save as docx.
3) open the round tripped file in MSO.

Expected behavior:
--------
The round-tripped file should open without any corruption.

Actual:
----
The roundtrip file is resulting in corruption.
Comment 1 Umesh Kadam 2014-03-13 09:23:51 UTC 
Created attachment 95700 [details]
File has comments for paragraphs.
Comment 2 Jorendc 2014-03-14 16:31:53 UTC 
Reproducible, tested using Mac OSX 10.9 with LibreOffice Version: 4.3.0.0.alpha0+
Build ID: 1b6e87fc232e78790d45a54b7a4ac7fe02d2c587
TinderBox: MacOSX-x86@49-TDF, Branch:master, Time: 2014-03-08_23:04:18

Kind regards,
Joren
Comment 3 Umesh Kadam 2014-03-17 18:03:16 UTC 
changes have been merged to master
Comment 4 Jorendc 2014-03-17 18:05:34 UTC 
Thanks for your patch :-)!

Looks like one of the bots didn't link your commit with bugzilla (maybe due the capital letters of FDO#<number>?).

Commit: http://cgit.freedesktop.org/libreoffice/core/commit/?id=ac5078a46632ef7efbdce9340c0b023d7f455d14

Thanks!
Comment 5 Umesh Kadam 2014-03-17 18:08:36 UTC 
changes have been merged to master
Comment 6 Jorendc 2014-03-17 18:09:51 UTC 
(In reply to comment #5)
> changes have been merged to master

Yes, so the 'target:4.3.0' in the whiteboard is correct :)?
Comment 7 Umesh Kadam 2014-03-17 18:24:06 UTC 
(In reply to comment #6)
> (In reply to comment #5)
> > changes have been merged to master
> 
> Yes, so the 'target:4.3.0' in the whiteboard is correct :)?
>
> Yes, the web page was not refreshed, I submitted it twice :)