25652 2009-05-08 13:58:38 -0700 wx port's PlatformPath may be deleted while in use 2009-05-08 15:53:59 -0700 1 1 1 Unclassified WebKit WebKit wx 528+ (Nightly build) PC All RESOLVED FIXED P2 Normal --- 0 kevinwatters webkit-unassigned oldest_to_newest 120464 0 kevinwatters 2009-05-08 13:58:38 -0700 PathWx.cpp's Path(const Path& path) constructor assigns m_path to another Path object's m_path. This is invalid since wx refcounts the underlying object and may delete it later. 120465 1 30140 kevinwatters 2009-05-08 14:01:09 -0700 Created attachment 30140 Use wxGraphicsPath copy constructor to avoid refcount bug Use wxGraphicsPath(const wxGraphicsPath&) instead of just stealing a reference to the object, so that we aren't bit by a segfault (or worse) later when the wxObject's refcount goes to zero. 120483 2 kevino 2009-05-08 15:53:59 -0700 Landed in r43418, thanks! 30140 2009-05-08 14:01:09 -0700 2009-05-08 15:49:46 -0700 Use wxGraphicsPath copy constructor to avoid refcount bug path-fix.patch text/plain 1257 kevinwatters SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0MzQwOCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTQgQEAKKzIwMDktMDUtMDggIEtldmluIFdhdHRlcnMgIDxrZXZpbndhdHRlcnNA Z21haWwuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg ICAgIE1ha2Ugc3VyZSB0aGUgcGF0aCdzIHJlZmRhdGEgaXNuJ3QgZGVsZXRlZCBiZWZvcmUgd2Un cmUgZG9uZSB3aXRoIHRoZSBvYmplY3QuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy93 eC9QYXRoV3guY3BwOgorICAgICAgICAoV2ViQ29yZTo6UGF0aDo6flBhdGgpOgorICAgICAgICAo V2ViQ29yZTo6UGF0aDo6UGF0aCk6CisgICAgICAgIChXZWJDb3JlOjpQYXRoOjp0cmFuc2xhdGUp OgorCiAyMDA5LTA1LTA4ICBFcmljIENhcmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUuY29tPgog CiAgICAgICAgIFJldmlld2VkIGJ5IERhcmluIEFkbGVyLgpJbmRleDogV2ViQ29yZS9wbGF0Zm9y bS9ncmFwaGljcy93eC9QYXRoV3guY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvcGxhdGZvcm0v Z3JhcGhpY3Mvd3gvUGF0aFd4LmNwcAkocmV2aXNpb24gNDM0MDgpCisrKyBXZWJDb3JlL3BsYXRm b3JtL2dyYXBoaWNzL3d4L1BhdGhXeC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTY2LDExICs2Niwx MiBAQAogCiBQYXRoOjp+UGF0aCgpCiB7CisgICAgY2xlYXIoKTsKIH0KIAogUGF0aDo6UGF0aChj b25zdCBQYXRoJiBwYXRoKQogeyAKLSAgICBtX3BhdGggPSAoUGxhdGZvcm1QYXRoKikmcGF0aC5t X3BhdGg7CisgICAgbV9wYXRoID0gbmV3IHd4R3JhcGhpY3NQYXRoKCpwYXRoLm1fcGF0aCk7CiB9 CiAKIGJvb2wgUGF0aDo6Y29udGFpbnMoY29uc3QgRmxvYXRQb2ludCYgcG9pbnQsIGNvbnN0IFdp bmRSdWxlIHJ1bGUpIGNvbnN0CkBAIC04OSw3ICs5MCw3IEBACiAKIHZvaWQgUGF0aDo6dHJhbnNs YXRlKGNvbnN0IEZsb2F0U2l6ZSYpCiB7IAotICAgIG5vdEltcGxlbWVudGVkKCk7IAorICAgIG5v dEltcGxlbWVudGVkKCk7CiB9CiAKIEZsb2F0UmVjdCBQYXRoOjpib3VuZGluZ1JlY3QoKSBjb25z dCAK