Issue 3672 in reviewboard: If 0C(in HEX) character in diff file, what been uploaded, then RB change it to newline, and its crashes the viewdiff

2014-11-17 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 3672 by golib...@gmail.com: If 0C(in HEX) character in diff  
file, what been uploaded, then RB change it to newline, and its crashes the  
viewdiff

https://code.google.com/p/reviewboard/issues/detail?id=3672

What version are you running?
2.0.11

What's the URL of the page containing the problem?
It's an internal page, i can't link it.

What steps will reproduce the problem?
1. A reviewrequest exists
2. Upload a new diff within the special character  (0C in HEX)
3. In diffview, the file not showing

What is the expected output? What do you see instead?
The expected output is to show the difference in viewdiff.

What operating system are you using? What browser?
Windows 7, Firefox/Chrome/IE8

Please provide any additional information below.
In the diff file what we uploading contains the  special character (0C  
in HEX) but when we download from RB it contains a new line where the  
special character was, and its corrupt the diff patch file.


In the RB's error dump (/tmp/reviewboard.ABC123/thefiles.diff), the special  
character also replaced to a new line character


Example
In the file we upload:
+/***/

In the file we can download:
+
/***/

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3672 in reviewboard: If 0C(in HEX) character in diff file, what been uploaded, then RB change it to newline, and its crashes the viewdiff

2014-11-17 Thread reviewboard

Updates:
Status: Duplicate
Mergedinto: 2916

Comment #1 on issue 3672 by trowb...@gmail.com: If 0C(in HEX) character  
in diff file, what been uploaded, then RB change it to newline, and its  
crashes the viewdiff

https://code.google.com/p/reviewboard/issues/detail?id=3672

(No comment was entered for this change.)

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.