buglet in aic7xxx.c?

[Mads Bondo Dydensborg]

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 at 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 at FreeBSD.org
with "unsubscribe aic7xxx" in the body of the message