On 20.01.2014 22:32, Lubomir Rintel wrote: > From: Gustavo Luiz Duarte <gustav...@linux.vnet.ibm.com> > > 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. > No longer necessarry ever since correct fix was committed > Link: https://bugzilla.redhat.com/show_bug.cgi?id=860834 > Link: http://lists.gnu.org/archive/html/grub-devel/2012-09/msg00081.html > > [lkund...@v3.sk: Add Changelog] > --- > ChangeLog | 6 ++++++ > grub-core/net/http.c | 2 +- > 2 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/ChangeLog b/ChangeLog > index 10abfe2..c91689e 100644 > --- a/ChangeLog > +++ b/ChangeLog > @@ -1,3 +1,9 @@ > +2014-01-20 Gustavo Luiz Duarte <gustav...@linux.vnet.ibm.com> > + > + * grub-core/net/http.c (http_establish): 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. > + > 2014-01-19 Colin Watson <cjwat...@ubuntu.com> > > * grub-core/osdep/freebsd/hostdisk.c (grub_util_fd_open): Ignore > diff --git a/grub-core/net/http.c b/grub-core/net/http.c > index 4684f8b..ef9538c 100644 > --- a/grub-core/net/http.c > +++ b/grub-core/net/http.c > @@ -393,7 +393,7 @@ http_establish (struct grub_file *file, grub_off_t > offset, int initial) > > data->sock = grub_net_tcp_open (file->device->net->server, > HTTP_PORT, http_receive, > - http_err, http_err, > + http_err, NULL, > file); > if (!data->sock) > { >
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Bug-grub mailing list Bug-grub@gnu.org https://lists.gnu.org/mailman/listinfo/bug-grub
