diff options
author | Michael Drake <tlsa@netsurf-browser.org> | 2014-01-24 14:21:27 +0000 |
---|---|---|
committer | Michael Drake <tlsa@netsurf-browser.org> | 2014-01-24 14:21:27 +0000 |
commit | 05dc1c1305e1de0fc77ac4671c336e96413911b0 (patch) | |
tree | 5a0bb15d6bac3579533552e39809f13aea842fdd /render/form.c | |
parent | 233904c7ed17002b434aa9c312976f2d4dfc08d9 (diff) | |
download | netsurf-05dc1c1305e1de0fc77ac4671c336e96413911b0.tar.gz netsurf-05dc1c1305e1de0fc77ac4671c336e96413911b0.tar.bz2 |
Fix Coverity-spotted issue. (Fixes behaviour for multiple selects.) Also minor optimisation.
Diffstat (limited to 'render/form.c')
-rw-r--r-- | render/form.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/render/form.c b/render/form.c index 5ce8056de..32c38be4b 100644 --- a/render/form.c +++ b/render/form.c @@ -1333,9 +1333,10 @@ static void form__select_process_selection(html_content *html, for (count = 0, o = control->data.select.items; o != NULL; count++, o = o->next) { - if (!control->data.select.multiple) + if (!control->data.select.multiple && o->selected) { o->selected = false; dom_html_option_element_set_selected(o->node, false); + } if (count == item) { if (control->data.select.multiple) { if (o->selected) { |