From 05dc1c1305e1de0fc77ac4671c336e96413911b0 Mon Sep 17 00:00:00 2001 From: Michael Drake Date: Fri, 24 Jan 2014 14:21:27 +0000 Subject: Fix Coverity-spotted issue. (Fixes behaviour for multiple selects.) Also minor optimisation. --- render/form.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'render/form.c') 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) { -- cgit v1.2.3