Merge branch 'mandersen-ios6-empty-node-crash' into develop

Cocoanetics · Feb 14, 2015 · feccd72 · feccd72
2 parents 620194a + 0deed07
commit feccd72
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/Core/Source/DTCoreTextParagraphStyle.m b/Core/Source/DTCoreTextParagraphStyle.m
@@ -43,7 +43,9 @@ + (DTCoreTextParagraphStyle *)paragraphStyleWithCTParagraphStyle:(CTParagraphSty
 #if DTCORETEXT_SUPPORT_NS_ATTRIBUTES
 + (DTCoreTextParagraphStyle *)paragraphStyleWithNSParagraphStyle:(NSParagraphStyle *)paragraphStyle
 {
-	NSParameterAssert(paragraphStyle);
+	if ( ! paragraphStyle) {
+		paragraphStyle = [NSParagraphStyle defaultParagraphStyle];
+	}
 
 	DTCoreTextParagraphStyle *retStyle = [[DTCoreTextParagraphStyle alloc] init];
 

diff --git a/Test/Source/NSAttributedStringHTMLTest.m b/Test/Source/NSAttributedStringHTMLTest.m
@@ -170,4 +170,16 @@ - (void)testAttributedStringColorToHTML
 }
  */
 
+- (void)testCrashAtEmptyNodeBeforeDivWithiOS6Attributes
+{
+	// This string is the simplest case that caused the crash.
+	NSString *html = @"<div><i></i><div></div></div>;";
+	NSData *data = [html dataUsingEncoding:NSUTF8StringEncoding];
+	NSDictionary *options = @{DTUseiOS6Attributes: @(YES)};
+	NSAttributedString *string = [[NSAttributedString alloc] initWithHTMLData:data
+																	  options:options
+														   documentAttributes:NULL];
+	XCTAssert(string != nil);
+}
+
 @end