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

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

Issue 2804483002: Check in the libxml roll script. (Closed)
Patch Set: Reenable SAX1; updater uses it. 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
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 *

Powered by Google App Engine
This is Rietveld 408576698