Skip to content

Commit

Permalink
Fix null pointer dereference in formUpdateBuffer
Browse files Browse the repository at this point in the history 
Bug-Debian: #28
  • Loading branch information
@tats
tats committed Oct 7, 2016
1 parent c92724e commit ec9eb22
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions form.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -459,6 +459,8 @@ formUpdateBuffer(Anchor *a, Buffer *buf, FormItemList *form)
}
else
#endif /* MENU_SELECT */
if (!form->value)
break;
p = form->value->ptr;
l = buf->currentLine;
if (!l)
Expand Down

2 comments on commit ec9eb22

@kcwu
Copy link
Contributor

@kcwu kcwu commented on ec9eb22 Oct 8, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This patch looks very weird.
If MENU_SELECT defined, line 464 is indented but not inside "else" block of line 460.

@tats
Copy link
Owner Author

@tats tats commented on ec9eb22 Oct 8, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, fixed by 22d29c3

Please sign in to comment.