[PATCH] Fix 2.4 EFI RTC oops

From: Bjorn Helgaas
Date: Wed Dec 17 2003 - 17:03:56 EST


This recent change to 2.4:

ChangeSet@xxxxxxxxxxxx, 2003-12-10 18:30:50-02:00, trini@xxxxxxxxxx
[PATCH] Fix rtc leak

===== arch/cris/drivers/ds1302.c 1.6 vs edited =====

broke efirtc.c (it causes a null pointer dereference on ia64).
The fix is below. Please apply.

Bjorn

P.S. The changeset above doesn't appear to be in 2.6 yet. If/when
you submit it for 2.6, you might consider using this style:

memset(&wtime, 0, sizeof(wtime));
memset(wtime, 0, sizeof(*wtime));

which is more obviously correct than:

memset(&wtime, 0, sizeof(struct rtc_time));
memset(wtime, 0, sizeof(struct rtc_time));


===== drivers/char/efirtc.c 1.6 vs edited =====
--- 1.6/drivers/char/efirtc.c Fri Oct 24 04:35:10 2003
+++ edited/drivers/char/efirtc.c Wed Dec 17 14:33:08 2003
@@ -118,7 +118,7 @@
static void
convert_from_efi_time(efi_time_t *eft, struct rtc_time *wtime)
{
- memset(&wtime, 0, sizeof(struct rtc_time));
+ memset(wtime, 0, sizeof(struct rtc_time));
wtime->tm_sec = eft->second;
wtime->tm_min = eft->minute;
wtime->tm_hour = eft->hour;

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/