vbatts/grub
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

net: Fix crash on http

Browse source 
Don't free file->data on receiving FIN flag since it is used all over
without checking. http_close() will be called later to free that memory.

Fixes bug: https://bugzilla.redhat.com/show_bug.cgi?id=860834

Signed-off-by: Gustavo Luiz Duarte <gustavold@linux.vnet.ibm.com>
Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
This commit is contained in:
Gustavo Luiz Duarte 2019-09-17 17:44:58 +02:00 committed by Daniel Kiper
parent 3ea9bc3c06
commit fc085f7f18
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
grub-core/net/http.c
View file

@ -392,7 +392,7 @@ http_establish (struct grub_file *file, grub_off_t offset, int initial)
data->sock = grub_net_tcp_open (file->device->net->server, data->sock = grub_net_tcp_open (file->device->net->server,
HTTP_PORT, http_receive, HTTP_PORT, http_receive,
http_err, http_err, http_err, NULL,
file); file);
if (!data->sock) if (!data->sock)
{ {