16842 2008-01-11 11:03:09 -0800 Hang in DRT in leaks mode due to signal handler doing unsafe things 2008-01-11 11:05:15 -0800 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Mac OS X 10.4 NEW P2 Normal --- 1 mrowe webkit-unassigned oldest_to_newest 66928 0 mrowe 2008-01-11 11:03:09 -0800 static void crashHandler(int sig) { fprintf(stderr, "%s\n", strsignal(sig)); restoreColorSpace(0); exit(128 + sig); } If "crashHandler" is triggered from within malloc/realloc/free then any calls to those functions may deadlock on a malloc lock. "restoreColorSpace" can run arbitrary system code, and "exit" will lead to C++ destructors being called on global objects. Both of these could result in a malloc-related function being called. 66929 1 18395 mrowe 2008-01-11 11:03:55 -0800 Created attachment 18395 Sample from buildbot showing this hang occurring 66932 2 mrowe 2008-01-11 11:05:15 -0800 I think that to call restoreColorSpace safely in the event of a crash it would need to be moved to a parent process that executes the real DRT. "_exit" can probably be used in place of "exit" to avoid the problems with that call. 18395 2008-01-11 11:03:55 -0800 2008-01-11 11:03:55 -0800 Sample from buildbot showing this hang occurring DumpRenderTree_19677.sample.txt text/plain 6321 mrowe QW5hbHlzaXMgb2Ygc2FtcGxpbmcgcGlkIDE5Njc3IGV2ZXJ5IDEwLjAwMDAwMCBtaWxsaXNlY29u ZHMKQ2FsbCBncmFwaDoKICAgIDEwMDAgVGhyZWFkXzEwMGIKICAgICAgMTAwMCBzdGFydAogICAg ICAgIDEwMDAgX3N0YXJ0CiAgICAgICAgICAxMDAwIG1haW4KICAgICAgICAgICAgMTAwMCBkdW1w UmVuZGVyVHJlZShpbnQsIGNoYXIgY29uc3QqKikKICAgICAgICAgICAgICAxMDAwIHJ1blRlc3Rp bmdTZXJ2ZXJMb29wKCkKICAgICAgICAgICAgICAgIDEwMDAgcnVuVGVzdChjaGFyIGNvbnN0KikK ICAgICAgICAgICAgICAgICAgMTAwMCAtW05TUnVuTG9vcCBydW5Nb2RlOmJlZm9yZURhdGU6XQog ICAgICAgICAgICAgICAgICAgIDEwMDAgQ0ZSdW5Mb29wUnVuSW5Nb2RlCiAgICAgICAgICAgICAg ICAgICAgICAxMDAwIENGUnVuTG9vcFJ1blNwZWNpZmljCiAgICAgICAgICAgICAgICAgICAgICAg IDEwMDAgX3NlbmRDYWxsYmFja3MKICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIC1bTlNV UkxDb25uZWN0aW9uKE5TVVJMQ29ubmVjdGlvbkludGVybmFsKSBfc2VuZENhbGxiYWNrc10KICAg ICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgLVtOU1VSTENvbm5lY3Rpb24oTlNVUkxDb25u ZWN0aW9uSW50ZXJuYWwpIF9zZW5kRGlkUmVjZWl2ZURhdGFDYWxsYmFja10KICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgMTAwMCAtW1dlYkNvcmVSZXNvdXJjZUhhbmRsZUFzRGVsZWdhdGUg Y29ubmVjdGlvbjpkaWRSZWNlaXZlRGF0YTpsZW5ndGhSZWNlaXZlZDpdCiAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgMTAwMCBXZWJDb3JlOjpSZXNvdXJjZUxvYWRlcjo6ZGlkUmVjZWl2 ZURhdGEoV2ViQ29yZTo6UmVzb3VyY2VIYW5kbGUqLCBjaGFyIGNvbnN0KiwgaW50LCBpbnQpCiAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIFdlYkNvcmU6Ok1haW5SZXNvdXJj ZUxvYWRlcjo6ZGlkUmVjZWl2ZURhdGEoY2hhciBjb25zdCosIGludCwgbG9uZyBsb25nLCBib29s KQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIFdlYkNvcmU6OlJlc291 cmNlTG9hZGVyOjpkaWRSZWNlaXZlRGF0YShjaGFyIGNvbnN0KiwgaW50LCBsb25nIGxvbmcsIGJv b2wpCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCBXZWJDb3JlOjpN YWluUmVzb3VyY2VMb2FkZXI6OmFkZERhdGEoY2hhciBjb25zdCosIGludCwgYm9vbCkKICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgV2ViQ29yZTo6RnJhbWVMb2Fk ZXI6OnJlY2VpdmVkRGF0YShjaGFyIGNvbnN0KiwgaW50KQogICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAxMDAwIFdlYkNvcmU6OkRvY3VtZW50TG9hZGVyOjpyZWNlaXZl ZERhdGEoY2hhciBjb25zdCosIGludCkKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAxMDAwIFdlYkNvcmU6OkRvY3VtZW50TG9hZGVyOjpjb21taXRMb2FkKGNoYXIg Y29uc3QqLCBpbnQpCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAxMDAwIFdlYkNvcmU6OkRvY3VtZW50TG9hZGVyOjpjb21taXRJZlJlYWR5KCkKICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCBXZWJDb3JlOjpGcmFt ZUxvYWRlcjo6Y29tbWl0UHJvdmlzaW9uYWxMb2FkKFdURjo6UGFzc1JlZlB0cjxXZWJDb3JlOjpD YWNoZWRQYWdlPikKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAxMDAwIFdlYkNvcmU6OkZyYW1lTG9hZGVyOjp0cmFuc2l0aW9uVG9Db21taXR0ZWQoV1RG OjpQYXNzUmVmUHRyPFdlYkNvcmU6OkNhY2hlZFBhZ2U+KQogICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCBXZWJGcmFtZUxvYWRlckNsaWVudDo6 dHJhbnNpdGlvblRvQ29tbWl0dGVkRm9yTmV3UGFnZSgpCiAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgLVtXZWJGcmFtZVZpZXcoV2ViSW50 ZXJuYWwpIF9tYWtlRG9jdW1lbnRWaWV3Rm9yRGF0YVNvdXJjZTpdCiAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCArW1dlYkZyYW1lVmll dyhXZWJJbnRlcm5hbCkgX3ZpZXdDbGFzc0Zvck1JTUVUeXBlOl0KICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgK1tXZWJWaWV3KFdl YlByaXZhdGUpIF92aWV3Q2xhc3M6YW5kUmVwcmVzZW50YXRpb25DbGFzczpmb3JNSU1FVHlwZTpd CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgIDEwMDAgK1tXZWJQREZWaWV3IHN1cHBvcnRlZE1JTUVUeXBlc10KICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwICtbV2Vi UERGUmVwcmVzZW50YXRpb24gc3VwcG9ydGVkTUlNRVR5cGVzXQogICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCAtW05TQXJy YXkgYXJyYXlCeUFkZGluZ09iamVjdHNGcm9tQXJyYXk6XQogICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIC1bTlNBcnJh eSBhcnJheUJ5QWRkaW5nT2JqZWN0czpjb3VudDpdCiAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCArW05TQXJyYXkg YXJyYXlXaXRoT2JqZWN0czpjb3VudDpdCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIC1bTlNQbGFjZWhvbGRl ckFycmF5IGluaXRXaXRoT2JqZWN0czpjb3VudDpdCiAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgQ0ZBcnJh eUNyZWF0ZQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgX19DRkFycmF5SW5pdAogICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgMTAwMCBfQ0ZSdW50aW1lQ3JlYXRlSW5zdGFuY2UKICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg MTAwMCBtYWxsb2Nfem9uZV9tYWxsb2MKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIGNoZWNr ZWRNYWxsb2MoX21hbGxvY196b25lX3QqLCB1bnNpZ25lZCBsb25nKQogICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgMTAwMCAweGU5YzQ3MDFlCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAg X3NpZ3RyYW1wCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCBpbnN0YWxsU2lnbmFs SGFuZGxlcnMoKQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCBleGl0CiAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgX19jeGFfZmluYWxpemUKICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIGN4YV9hdGV4aXRfd3JhcHBlcgogICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCBfX3RjZl8xCiAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgIDEwMDAgV1RGOjpIYXNoU2V0PEtKUzo6SlNPYmplY3Qq LCBXVEY6OlB0ckhhc2g8S0pTOjpKU09iamVjdCo+LCBXVEY6Okhhc2hUcmFpdHM8S0pTOjpKU09i amVjdCo+ID46On5IYXNoU2V0IFtpbi1jaGFyZ2VdKCkKICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAxMDAwIFdURjo6SGFzaFRhYmxlPGludCwgaW50LCBXVEY6OklkZW50 aXR5RXh0cmFjdG9yPGludD4sIFdURjo6SW50SGFzaDxpbnQ+LCBXVEY6Okhhc2hUcmFpdHM8aW50 PiwgV1RGOjpIYXNoVHJhaXRzPGludD4gPjo6fkhhc2hUYWJsZSBbaW4tY2hhcmdlXSgpCiAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIFdURjo6SGFzaFRhYmxl PGludCwgaW50LCBXVEY6OklkZW50aXR5RXh0cmFjdG9yPGludD4sIFdURjo6SW50SGFzaDxpbnQ+ LCBXVEY6Okhhc2hUcmFpdHM8aW50PiwgV1RGOjpIYXNoVHJhaXRzPGludD4gPjo6ZGVhbGxvY2F0 ZVRhYmxlKGludCosIGludCkKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgMTAwMCBXVEY6OmZhc3RGcmVlKHZvaWQqKQogICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgIDEwMDAgZnJlZQogICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgMTAwMCBfX3NwaW5fbG9jawogICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAxMDAwIF9fc3Bpbl9sb2NrCiAgICAx MDAwIFRocmVhZF8xMTAzCiAgICAgIDEwMDAgX3B0aHJlYWRfYm9keQogICAgICAgIDEwMDAgZm9y a1RocmVhZEZvckZ1bmN0aW9uCiAgICAgICAgICAxMDAwICtbTlNVUkxDYWNoZSBfZGlza0NhY2hl U3luY0xvb3A6XQogICAgICAgICAgICAxMDAwIENGUnVuTG9vcFJ1bkluTW9kZQogICAgICAgICAg ICAgIDEwMDAgQ0ZSdW5Mb29wUnVuU3BlY2lmaWMKICAgICAgICAgICAgICAgIDEwMDAgbWFjaF9t c2dfdHJhcAogICAgICAgICAgICAgICAgICAxMDAwIG1hY2hfbXNnX3RyYXAKICAgIDEwMDAgVGhy ZWFkXzEyMDMKICAgICAgMTAwMCBfcHRocmVhZF9ib2R5CiAgICAgICAgMTAwMCBmb3JrVGhyZWFk Rm9yRnVuY3Rpb24KICAgICAgICAgIDEwMDAgK1tOU1VSTENvbm5lY3Rpb24oTlNVUkxDb25uZWN0 aW9uSW50ZXJuYWwpIF9yZXNvdXJjZUxvYWRMb29wOl0KICAgICAgICAgICAgMTAwMCBDRlJ1bkxv b3BSdW5Jbk1vZGUKICAgICAgICAgICAgICAxMDAwIENGUnVuTG9vcFJ1blNwZWNpZmljCiAgICAg ICAgICAgICAgICAxMDAwIF9DRlJlbGVhc2UKICAgICAgICAgICAgICAgICAgMTAwMCBfX3NwaW5f bG9jawogICAgICAgICAgICAgICAgICAgIDEwMDAgX19zcGluX2xvY2sKClRvdGFsIG51bWJlciBp biBzdGFjayAocmVjdXJzaXZlIGNvdW50ZWQgbXVsdGlwbGUsIHdoZW4gPj01KToKClNvcnQgYnkg dG9wIG9mIHN0YWNrLCBzYW1lIGNvbGxhcHNlZCAod2hlbiA+PSA1KToKICAgICAgICBfX3NwaW5f bG9jayAgICAgICAgMjAwMAogICAgICAgIG1hY2hfbXNnX3RyYXAgICAgICAgIDEwMDAK