Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(174)

Unified Diff: third_party/libxml/chromium/chromium-issue-599427.patch

Issue 2804483002: Check in the libxml roll script. (Closed)
Patch Set: Code review feedback for the roll script. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/libxml/README.chromium ('k') | third_party/libxml/chromium/chromium-issue-620679.patch » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/libxml/chromium/chromium-issue-599427.patch
diff --git a/third_party/libxml/chromium/chromium-issue-599427.patch b/third_party/libxml/chromium/chromium-issue-599427.patch
new file mode 100644
index 0000000000000000000000000000000000000000..20e26c51942aa386b45d737d3b1cb7e4d69fc868
--- /dev/null
+++ b/third_party/libxml/chromium/chromium-issue-599427.patch
@@ -0,0 +1,28 @@
+--- a/xmlstring.c 2016-05-13 17:24:58.870079372 +0900
++++ b/xmlstring.c 2016-04-25 13:58:09.879238595 +0900
+@@ -835,16 +835,20 @@
+ while ( len-- > 0) {
+ if ( !*ptr )
+ break;
+- if ( (ch = *ptr++) & 0x80)
+- while ((ch<<=1) & 0x80 ) {
+- if (*ptr == 0) break;
++ if ( (ch = *ptr++) & 0x80) {
++ // Workaround for an optimization bug in VS 2015 Update 2, remove
++ // once the fix is released. crbug.com/599427
++ // https://connect.microsoft.com/VisualStudio/feedback/details/2582138
++ xmlChar ch2 = ch;
++ while ((ch2<<=1) & 0x80 ) {
+ ptr++;
+- }
++ if (*ptr == 0) break;
++ }
++ }
+ }
+ return (ptr - utf);
+ }
+
+-
+ /**
+ * xmlUTF8Strndup:
+ * @utf: the input UTF8 *
« no previous file with comments | « third_party/libxml/README.chromium ('k') | third_party/libxml/chromium/chromium-issue-620679.patch » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698