[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
gEDA-cvs: gaf.git: branch: master updated (1.7.1-20110619-266-gc63deda)
The branch, master has been updated
via c63deda32d9480e7b5b5597d3fb1fed8e6591805 (commit)
from 56c72d5f24d6c80d47d6c114a1bcae1156c68ce6 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
=========
Summary
=========
libgeda/src/o_picture.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
=================
Commit Messages
=================
commit c63deda32d9480e7b5b5597d3fb1fed8e6591805
Author: Peter TB Brett <peter@xxxxxxxxxxxxx>
Commit: Peter TB Brett <peter@xxxxxxxxxxxxx>
libgeda: Fix null dereference when fallback image not loaded.
In commit bccf15c104d6, a bug was introduced whereby libgeda
dereferenced a NULL pointer. If a picture object's image cannot be
loaded, libgeda attempts to load a fallback image, and on failure,
accesses a GError to retrieve the error message. Unfortunately, the
GError was not being passed to the image loading function, and was
therefore uninitialised when it was dereferenced to obtain a message.
Closes-bug: lp-904639
:100644 100644 82a5bac... 43b1124... M libgeda/src/o_picture.c
=========
Changes
=========
commit c63deda32d9480e7b5b5597d3fb1fed8e6591805
Author: Peter TB Brett <peter@xxxxxxxxxxxxx>
Commit: Peter TB Brett <peter@xxxxxxxxxxxxx>
libgeda: Fix null dereference when fallback image not loaded.
In commit bccf15c104d6, a bug was introduced whereby libgeda
dereferenced a NULL pointer. If a picture object's image cannot be
loaded, libgeda attempts to load a fallback image, and on failure,
accesses a GError to retrieve the error message. Unfortunately, the
GError was not being passed to the image loading function, and was
therefore uninitialised when it was dereferenced to obtain a message.
Closes-bug: lp-904639
diff --git a/libgeda/src/o_picture.c b/libgeda/src/o_picture.c
index 82a5bac..43b1124 100644
--- a/libgeda/src/o_picture.c
+++ b/libgeda/src/o_picture.c
@@ -1231,11 +1231,11 @@ o_picture_get_fallback_pixbuf (TOPLEVEL *toplevel)
filename = g_build_filename (toplevel->bitmap_directory,
"gschem-warning.png", NULL);
- pixbuf = gdk_pixbuf_new_from_file (filename, NULL);
+ pixbuf = gdk_pixbuf_new_from_file (filename, &error);
if (pixbuf == NULL) {
g_warning ( _("Failed to load fallback image %s: %s.\n"),
- filename, error->message);
+ filename, error->message);
g_error_free (error);
failed = TRUE;
}
_______________________________________________
geda-cvs mailing list
geda-cvs@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-cvs