Fix build breakage in gdb/xml-support.c
The buildbots are showing that the previous change to xml_fetch_content_from_file causes __wur warnings/errors: ../../binutils-gdb/gdb/xml-support.c: In function gdb::unique_xmalloc_ptr<char> xml_fetch_content_from_file(const char*, void*): ../../binutils-gdb/gdb/xml-support.c:1028:43: error: ignoring return value of size_t fread(void*, size_t, size_t, FILE*), declared with attribute warn_unused_result [-Werror=unused-result] fread (text.get (), 1, len, file.get ()); ^ This commit fixes it. gdb/ChangeLog: 2017-10-19 Pedro Alves <palves@redhat.com> * xml-support.c (xml_fetch_content_from_file): Check fread's return.
This commit is contained in:
parent
9d06997adb
commit
63929e843d
|
@ -1,3 +1,8 @@
|
|||
2017-10-19 Pedro Alves <palves@redhat.com>
|
||||
|
||||
* xml-support.c (xml_fetch_content_from_file): Check fread's
|
||||
return.
|
||||
|
||||
2017-10-19 Pedro Alves <palves@redhat.com>
|
||||
|
||||
* ser-base.c (ser_base_read_error_fd): Delete the file handler if
|
||||
|
|
|
@ -1025,8 +1025,8 @@ xml_fetch_content_from_file (const char *filename, void *baton)
|
|||
|
||||
gdb::unique_xmalloc_ptr<char> text ((char *) xmalloc (len + 1));
|
||||
|
||||
fread (text.get (), 1, len, file.get ());
|
||||
if (ferror (file.get ()))
|
||||
if (fread (text.get (), 1, len, file.get ()) != len
|
||||
|| ferror (file.get ()))
|
||||
{
|
||||
warning (_("Read error from \"%s\""), filename);
|
||||
return NULL;
|
||||
|
|
Loading…
Reference in New Issue