Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 29 Sep 2000 12:55:56 +0200 (CEST)
From:      Mads Bondo Dydensborg <madsdyd@challenge.dk>
To:        "'aic7xxx@FreeBSD.ORG'" <aic7xxx@FreeBSD.ORG>
Subject:   buglet in aic7xxx.c?
Message-ID:  <Pine.LNX.4.21.0009291252490.32689-100000@challenge.dk>

next in thread | raw e-mail | index | archive | help
Hi there.

Just saw this snippet of code in 
static int
aic7xxx_register(Scsi_Host_Template *template, struct aic7xxx_host *p,
  int reset_delay)

driver version 5.1.20/linux kernel 2.2.13

----------

  /*
   * Lock out other contenders for our i/o space.
   */
  request_region(p->base, MAXREG - MINREG, "aic7xxx");

----------

Can't request_region fail? Shouldn't the return value be checked?

Mads

-- 
Mads Bondo Dydensborg.                               madsdyd@challenge.dk
Smoking kills. If you're killed, you've lost a very important part of your
life.  
            - Brooke Shields, during an interview to become spokesperson for a
                                               federal anti-smoking campaign.



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe aic7xxx" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.LNX.4.21.0009291252490.32689-100000>