Re: [PATCH 05/18] KDrive: Bail out if screen initialization failed

2013-04-08 Thread Keith Packard
Geert Uytterhoeven  writes:

> Else we may get a segmentation fault later.

Reviewed-by: Keith Packard 

-- 
keith.pack...@intel.com


pgpoQ8KPKYy6p.pgp
Description: PGP signature


[PATCH 05/18] KDrive: Bail out if screen initialization failed

2013-03-27 Thread Geert Uytterhoeven
Else we may get a segmentation fault later.

Signed-off-by: Geert Uytterhoeven 
---
 hw/kdrive/src/kdrive.c |3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/hw/kdrive/src/kdrive.c b/hw/kdrive/src/kdrive.c
index 716f18e..84d39bb 100644
--- a/hw/kdrive/src/kdrive.c
+++ b/hw/kdrive/src/kdrive.c
@@ -943,7 +943,8 @@ KdInitScreen(ScreenInfo * pScreenInfo,
 {
 KdCardInfo *card = screen->card;
 
-(*card->cfuncs->scrinit) (screen);
+if (!(*card->cfuncs->scrinit) (screen))
+FatalError("Screen initialization failed!\n");
 
 if (!card->cfuncs->initAccel)
 screen->dumb = TRUE;
-- 
1.7.0.4


-- 
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/1364390451-9656-6-git-send-email-ge...@linux-m68k.org