105867 2012-12-30 03:20:56 -0800 [EFL][WK2] Invalid use of temporary object in ProcessLauncher::launchProcess() 2012-12-30 05:04:24 -0800 1 1 1 Unclassified WebKit WebKit EFL 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 105868 1 cdumez cdumez gyuyoung.kim kangil.webkit kenneth laszlo.gombos lucas.de.marchi rakuco ryuan.choi tmpsantos webkit.review.bot oldest_to_newest 798205 0 cdumez 2012-12-30 03:20:56 -0800 A pointer to the data representation of a temporary object is stored in prefixedExecutablePath. 798206 1 180952 cdumez 2012-12-30 03:30:29 -0800 Created attachment 180952 Patch 798207 2 180952 webkit.review.bot 2012-12-30 03:57:55 -0800 Comment on attachment 180952 Patch Clearing flags on attachment: 180952 Committed r138574: <http://trac.webkit.org/changeset/138574> 798208 3 webkit.review.bot 2012-12-30 03:58:00 -0800 All reviewed patches have been landed. Closing bug. 798210 4 180952 tmpsantos 2012-12-30 04:44:43 -0800 Comment on attachment 180952 Patch What is this patch fixing? I failed to see the illegal assignment. 798212 5 180952 cdumez 2012-12-30 04:45:58 -0800 Comment on attachment 180952 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=180952&action=review > Source/WebKit2/UIProcess/Launcher/efl/ProcessLauncherEfl.cpp:-68 > - prefixedExecutablePath = prefixedExecutablePathStr.utf8().data(); This line is the what I'm fixing. 180952 2012-12-30 03:30:29 -0800 2012-12-30 04:45:58 -0800 Patch 105867_launchProcess_temp.patch text/plain 2303 cdumez ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCA4ZWIyZmY2Li5hNTUyYzRmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYg QEAKKzIwMTItMTItMzAgIENocmlzdG9waGUgRHVtZXogIDxjaHJpc3RvcGhlLmR1bWV6QGludGVs LmNvbT4KKworICAgICAgICBbRUZMXVtXSzJdIEludmFsaWQgdXNlIG9mIHRlbXBvcmFyeSBvYmpl Y3QgaW4gUHJvY2Vzc0xhdW5jaGVyOjpsYXVuY2hQcm9jZXNzKCkKKyAgICAgICAgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEwNTg2NworCisgICAgICAgIFJldmlld2Vk IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEZpeCBpbnZhbGlkIHVzZSBvZiB0ZW1wb3Jh cnkgb2JqZWN0IGluIFByb2Nlc3NMYXVuY2hlcjo6bGF1bmNoUHJvY2VzcygpCisgICAgICAgIGZv ciBwcmVmaXhlZEV4ZWN1dGFibGVQYXRoLgorCisgICAgICAgICogVUlQcm9jZXNzL0xhdW5jaGVy L2VmbC9Qcm9jZXNzTGF1bmNoZXJFZmwuY3BwOgorICAgICAgICAoV2ViS2l0OjpQcm9jZXNzTGF1 bmNoZXI6OmxhdW5jaFByb2Nlc3MpOgorCiAyMDEyLTEyLTI5ICBTYW0gV2VpbmlnICA8c2FtQHdl YmtpdC5vcmc+CiAKICAgICAgICAgV2luZG93cyBidWlsZCBmaXguCmRpZmYgLS1naXQgYS9Tb3Vy Y2UvV2ViS2l0Mi9VSVByb2Nlc3MvTGF1bmNoZXIvZWZsL1Byb2Nlc3NMYXVuY2hlckVmbC5jcHAg Yi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvTGF1bmNoZXIvZWZsL1Byb2Nlc3NMYXVuY2hlckVm bC5jcHAKaW5kZXggMmIzMDg5Ni4uY2RkMDYwMSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIv VUlQcm9jZXNzL0xhdW5jaGVyL2VmbC9Qcm9jZXNzTGF1bmNoZXJFZmwuY3BwCisrKyBiL1NvdXJj ZS9XZWJLaXQyL1VJUHJvY2Vzcy9MYXVuY2hlci9lZmwvUHJvY2Vzc0xhdW5jaGVyRWZsLmNwcApA QCAtNjEsMTEgKzYxLDkgQEAgdm9pZCBQcm9jZXNzTGF1bmNoZXI6OmxhdW5jaFByb2Nlc3MoKQog ICAgIHNucHJpbnRmKHNvY2tldCwgc2l6ZW9mKHNvY2tldCksICIlZCIsIHNvY2tldHNbMF0pOwog CiAjaWZuZGVmIE5ERUJVRwotICAgIGNvbnN0IGNoYXIqIHByZWZpeGVkRXhlY3V0YWJsZVBhdGgg PSAwOwotICAgIFN0cmluZyBwcmVmaXhlZEV4ZWN1dGFibGVQYXRoU3RyOworICAgIFN0cmluZyBw cmVmaXhlZEV4ZWN1dGFibGVQYXRoOwogICAgIGlmICghbV9sYXVuY2hPcHRpb25zLnByb2Nlc3ND bWRQcmVmaXguaXNFbXB0eSgpKSB7Ci0gICAgICAgIHByZWZpeGVkRXhlY3V0YWJsZVBhdGhTdHIg PSBtX2xhdW5jaE9wdGlvbnMucHJvY2Vzc0NtZFByZWZpeCArICcgJyArIGV4ZWN1dGFibGVQYXRo LmRhdGEoKSArICcgJyArIHNvY2tldDsKLSAgICAgICAgcHJlZml4ZWRFeGVjdXRhYmxlUGF0aCA9 IHByZWZpeGVkRXhlY3V0YWJsZVBhdGhTdHIudXRmOCgpLmRhdGEoKTsKKyAgICAgICAgcHJlZml4 ZWRFeGVjdXRhYmxlUGF0aCA9IG1fbGF1bmNoT3B0aW9ucy5wcm9jZXNzQ21kUHJlZml4ICsgJyAn ICsgU3RyaW5nOjpmcm9tVVRGOChleGVjdXRhYmxlUGF0aC5kYXRhKCkpICsgJyAnICsgc29ja2V0 OwogICAgIH0KICNlbmRpZgogCkBAIC03NiwxMSArNzQsMTEgQEAgdm9pZCBQcm9jZXNzTGF1bmNo ZXI6OmxhdW5jaFByb2Nlc3MoKQogICAgIGlmICghcGlkKSB7IC8vIENoaWxkIHByb2Nlc3MuCiAg ICAgICAgIGNsb3NlKHNvY2tldHNbMV0pOwogI2lmbmRlZiBOREVCVUcKLSAgICAgICAgaWYgKHBy ZWZpeGVkRXhlY3V0YWJsZVBhdGgpIHsKKyAgICAgICAgaWYgKCFwcmVmaXhlZEV4ZWN1dGFibGVQ YXRoLmlzTnVsbCgpKSB7CiAgICAgICAgICAgICAvLyBGSVhNRTogVGhpcyBpcyBub3QgY29ycmVj dCBiZWNhdXNlIGl0IGludm9rZXMgdGhlIHNoZWxsCiAgICAgICAgICAgICAvLyBhbmQga2VlcHMg dGhpcyBwcm9jZXNzIHdhaXRpbmcuIFNob3VsZCBiZSBjaGFuZ2VkIHRvCiAgICAgICAgICAgICAv LyBzb21ldGhpbmcgbGlrZSBleGVjdnAoKS4KLSAgICAgICAgICAgIGlmIChzeXN0ZW0ocHJlZml4 ZWRFeGVjdXRhYmxlUGF0aCkgPT0gLTEpIHsKKyAgICAgICAgICAgIGlmIChzeXN0ZW0ocHJlZml4 ZWRFeGVjdXRhYmxlUGF0aC51dGY4KCkuZGF0YSgpKSA9PSAtMSkgewogICAgICAgICAgICAgICAg IEFTU0VSVF9OT1RfUkVBQ0hFRCgpOwogICAgICAgICAgICAgICAgIGV4aXQoRVhJVF9GQUlMVVJF KTsKICAgICAgICAgICAgIH0gZWxzZQo=