From 00decd3b6cea0de22c88b9504dbe26d680e8ab16 Mon Sep 17 00:00:00 2001 From: Krzysztof Halasa Date: Sat, 27 Jan 2007 12:29:31 -0800 Subject: [PATCH] Don't zero out mode structure after setting name field. Instead of using memset to zero the structure (at the wrong time, no less), use xcalloc to gather pre-initialized memory. --- src/i830_xf86Modes.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/i830_xf86Modes.c b/src/i830_xf86Modes.c index f37eaeb7..b5f06d6c 100644 --- a/src/i830_xf86Modes.c +++ b/src/i830_xf86Modes.c @@ -564,7 +564,7 @@ xf86GetConfigModes (XF86ConfModeLinePtr conf_mode) for (; conf_mode; conf_mode = (XF86ConfModeLinePtr) conf_mode->list.next) { - mode = xalloc(sizeof(DisplayModeRec)); + mode = xcalloc(1, sizeof(DisplayModeRec)); if (!mode) continue; mode->name = xstrdup(conf_mode->ml_identifier); @@ -573,8 +573,6 @@ xf86GetConfigModes (XF86ConfModeLinePtr conf_mode) xfree (mode); continue; } - - memset(mode,'\0',sizeof(DisplayModeRec)); mode->type = 0; mode->Clock = conf_mode->ml_clock; mode->HDisplay = conf_mode->ml_hdisplay;