<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto">I think that perhaps we have a problem with the retain count flags in the definition of CFSTR. Previously the pinned reference would prevent the deallocation. This may have gotten clobbered with the latest CF import.<br><br><div id="AppleMailSignature">Sent from my iPhone</div><div><br>On Nov 29, 2017, at 7:55 PM, Brandon Williams via swift-corelibs-dev <<a href="mailto:swift-corelibs-dev@swift.org">swift-corelibs-dev@swift.org</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr"><div>Hey Tony,</div><div><br></div><div>We were able to fix our crashes by avoiding our uses of `URLQueryItem`s. In one place I had to completely remove the use of `URLQueryItem`:</div><div><br></div><div><a href="https://github.com/pointfreeco/swift-web/pull/79/files#diff-293b73a926ba418bd9511ef735fc947cL147" target="_blank">https://github.com/pointfreeco/swift-web/pull/79/files#diff-293b73a926ba418bd9511ef735fc947cL147</a><br></div><div><br></div><div>And in another my colleague Stephen Celis discovered that duplicate keys in a query string can cause this to crash:</div><div><br></div><div><a href="https://github.com/pointfreeco/swift-web/pull/80">https://github.com/pointfreeco/swift-web/pull/80</a><br></div><div><br></div><div>He was able to come up with a minimal test case to demonstrate this:</div><div><br></div><a href="https://github.com/pointfreeco/swift-web/commit/cfa9b519a829f1c1627620521326f0877c2d023a">https://github.com/pointfreeco/swift-web/commit/cfa9b519a829f1c1627620521326f0877c2d023a</a><div><br></div><div>And you can see the failure in this Travis CI report:</div><div><br></div><div><a href="https://travis-ci.org/pointfreeco/swift-web/builds/309316418?utm_source=email&utm_medium=notification">https://travis-ci.org/pointfreeco/swift-web/builds/309316418</a></div><div><br></div><div>At the bottom you’ll find:</div><div><br></div><div><div>```fatal error: Constant strings cannot be deallocated: file Foundation/NSCFString.swift, line 118```</div><div><br></div><div>So definitely seems to be in that area!</div><div><br></div><div>Now, as far as rebuilding swift-corelibs-foundation, I’m down to try but I dont really know much about how to do that. With some instructions I could give it a shot.</div><div><br></div><div>Thanks for the help!</div><div><br></div><div><br></div></div><div><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 29, 2017 at 5:59 PM Tony Parker <<a href="mailto:anthony.parker@apple.com" target="_blank">anthony.parker@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">I see a couple of places that are suspicious there:<div><br></div><div>Description:</div><div><br></div><div><a href="https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L66" target="_blank">https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L66</a></div><div><br></div><div>And the nameString / valueString constants in the copy query items function:</div><div><br></div><div><a href="https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1057" target="_blank">https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1057</a></div><div><a href="https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1079" target="_blank">https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1079</a></div><div><a href="https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1104" target="_blank">https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1104</a></div><div><a href="https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1134" target="_blank">https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1134</a></div><div><a href="https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1158" target="_blank">https://github.com/apple/swift-corelibs-foundation/blob/master/CoreFoundation/URL.subproj/CFURLComponents.c#L1158</a></div><div><br></div><div>Do you have the ability to re-build swift-corelibs-foundation and run a test? I suspect if you replace those with CFRetain(CFSTR(…)) then it will fix your crash.</div><div><br></div><div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div>- Tony</div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><br><div><br><blockquote type="cite"><div>On Nov 29, 2017, at 2:06 PM, Brandon Williams <<a href="mailto:mbw234@gmail.com" target="_blank">mbw234@gmail.com</a>> wrote:</div><br class="m_-2036326282292254873m_435483929937444131Apple-interchange-newline"><div><div dir="ltr">Thanks for the info!<div><br></div><div>FWIW, most of the places I have encountered this so far have made use of `URLComponents`, and have even been able to eliminate the crash by getting rid of that code in a few places. The other JIRA bug on this topic also mentions URLComponents in their repro case. Seems to be the culprit.</div><div><br></div><div>Also worth noting that in the above cases dealing with URLComponents the crash only happens in DEBUG compilations, not RELEASE.</div><div><br></div><div>However, I do have another one of these crashes that _does_ happen on RELEASE builds that I haven’t yet been able to reduce.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 29, 2017 at 4:43 PM Tony Parker <<a href="mailto:anthony.parker@apple.com" target="_blank">anthony.parker@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">Hi Brandon,<div><br></div><div>This is probably a bug in the CoreFoundation C sources part of swift-corelibs-foundation. Unlike Darwin platforms, we can’t make the CFSTR(“”) macro produce a CFStringRef that cannot be overreleased. Probably there is a constant string returned from CF function, then the Swift runtime assumes it can release it, and bam we wind up with an overrelease.</div><div><br></div><div>We’ll have to track down which function is getting called, which probably just requires stepping through that test case with a debugger…</div><div><br></div><div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div>- Tony<br></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><div><blockquote type="cite"><div>On Nov 29, 2017, at 1:34 PM, Brandon Williams via swift-corelibs-dev <<a href="mailto:swift-corelibs-dev@swift.org" target="_blank">swift-corelibs-dev@swift.org</a>> wrote:</div><br class="m_-2036326282292254873m_435483929937444131m_-9212223342634817513Apple-interchange-newline"></blockquote></div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><div><blockquote type="cite"><div><div dir="ltr"><div>Hello all!</div><div><br></div><div>We’ve been encountering this runtime error quite a bit recently, and we have no idea why. We’ve filed a bug and there’s one other on JIRA related to this, but both without any comments:</div><div><br></div><a href="https://bugs.swift.org/browse/SR-6422" target="_blank">https://bugs.swift.org/browse/SR-6422</a><br><div><a href="https://bugs.swift.org/browse/SR-6398" target="_blank">https://bugs.swift.org/browse/SR-6398</a><br></div><div><br></div><div>We’re curious if others are aware of this and or has more info? Is it a red herring for some other problem?</div><div><br></div><div>Any info would be helpful!</div></div></div></blockquote></div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><div><blockquote type="cite"><div>
_______________________________________________<br>swift-corelibs-dev mailing list<br><a href="mailto:swift-corelibs-dev@swift.org" target="_blank">swift-corelibs-dev@swift.org</a><br><a href="https://lists.swift.org/mailman/listinfo/swift-corelibs-dev" target="_blank">https://lists.swift.org/mailman/listinfo/swift-corelibs-dev</a><br></div></blockquote></div></div></div></blockquote></div>
</div></blockquote></div><br></div></div></blockquote></div></div></div>
</div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>swift-corelibs-dev mailing list</span><br><span><a href="mailto:swift-corelibs-dev@swift.org">swift-corelibs-dev@swift.org</a></span><br><span><a href="https://lists.swift.org/mailman/listinfo/swift-corelibs-dev">https://lists.swift.org/mailman/listinfo/swift-corelibs-dev</a></span><br></div></blockquote></body></html>