11437 2006-10-27 11:58:23 -0700 use of uninitialized variable 2006-10-28 12:07:41 -0700 1 1 1 Unclassified WebKit HTML Editing 420+ Mac OS X 10.4 RESOLVED FIXED http://www.google.com P2 Normal --- 1 dacarson dacarson oldest_to_newest 49299 0 dacarson 2006-10-27 11:58:23 -0700 In the function SelectionController::setModifyBias(), the case EXTEND: is be hit before m_modifyBiasSet has been initialized. The case funciton looks like this: case EXTEND: if (!m_modifyBiasSet) { m_modifyBiasSet = true; .... It is being hit when the user first starts to type something into the search box on www.google.com. Not sure what the value should be initialized to. 49306 1 dacarson 2006-10-27 13:27:44 -0700 From the looks of the code, it seems as if the value should be initialized to false. 49308 2 11259 dacarson 2006-10-27 13:36:02 -0700 Created attachment 11259 patch to init value to false 49251 3 11259 beidson 2006-10-27 13:55:40 -0700 Comment on attachment 11259 patch to init value to false r=me 48839 4 sam 2006-10-28 12:07:41 -0700 Landed in r17400. 11259 2006-10-27 13:36:02 -0700 2006-10-27 13:55:40 -0700 patch to init value to false initVar.txt text/plain 1068 dacarson SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAxNzM4NSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTQgQEAKKzIwMDYtMTAtMjcgIGRhY2Fyc29uICBkYWNhcnNvbkBnbWFpbC5jb20K KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGaXggZm9y OiBodHRwOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTQzNworICAgICAgICBJ bml0aWFsaXplIG1fbW9kaWZ5Qmlhc1NldCB0byBmYWxzZSwgYXMgaXQgc29tZXRpbWVzIGdldHMg CisgICAgICAgIHVzZWQgYmVmb3JlIGl0IGhhcyBiZWVuIGluaXRhbGl6ZWQuCisKKyAgICAgICAg KiBlZGl0aW5nL1NlbGVjdGlvbkNvbnRyb2xsZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6U2Vs ZWN0aW9uQ29udHJvbGxlcjo6U2VsZWN0aW9uQ29udHJvbGxlcik6CisKIDIwMDYtMTAtMjcgIEdl b2ZmcmV5IEdhcmVuICA8Z2dhcmVuQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBC cmFkeS4KSW5kZXg6IFdlYkNvcmUvZWRpdGluZy9TZWxlY3Rpb25Db250cm9sbGVyLmNwcAo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBXZWJDb3JlL2VkaXRpbmcvU2VsZWN0aW9uQ29udHJvbGxlci5jcHAJKHJldmlz aW9uIDE3MzY2KQorKysgV2ViQ29yZS9lZGl0aW5nL1NlbGVjdGlvbkNvbnRyb2xsZXIuY3BwCSh3 b3JraW5nIGNvcHkpCkBAIC00OCw2ICs0OCw3IEBAIFNlbGVjdGlvbkNvbnRyb2xsZXI6OlNlbGVj dGlvbkNvbnRyb2xsZXIKICAgICA6IG1fbmVlZHNMYXlvdXQodHJ1ZSkKICAgICAsIG1fZnJhbWUo ZnJhbWUpCiAgICAgLCBtX2lzRHJhZ0NhcmV0Q29udHJvbGxlcihpc0RyYWdDYXJldENvbnRyb2xs ZXIpCisgICAgLCBtX21vZGlmeUJpYXNTZXQoZmFsc2UpCiB7CiB9CiAK