Messages in this thread | | | Date | Mon, 01 Sep 2008 02:15:05 +0900 (JST) | Subject | Re: [PATCH 1/2] ne.c fix, res->name | From | Atsushi Nemoto <> |
| |
On Sat, 30 Aug 2008 13:05:03 -0500, David Fries <david@fries.net> wrote: > On Sun, Aug 31, 2008 at 12:03:57AM +0900, Atsushi Nemoto wrote: > > On Fri, 29 Aug 2008 21:44:25 -0500, David Fries <david@fries.net> wrote: > > > + if (!strcmp(res->name, BAD_STR)) > > > + dev->mem_end = BAD; > > > > The res->name might be NULL. You should check it first. > > > > if (res->name && !strcmp(res->name, BAD_STR)) > > dev->mem_end = BAD; > > The name gets a valid string by that point in time. > > drivers/base/platform.c:257
Oh, you are right. Sorry for noise.
--- Atsushi Nemoto
| |