Bug 5681 - can fail with "bind interfaces only" and >128 IP addresses
can fail with "bind interfaces only" and >128 IP addresses
Status: RESOLVED FIXED
Product: Samba 3.2
Classification: Unclassified
Component: File services
3.2.0
Other Linux
: P3 minor
: ---
Assigned To: Jeremy Allison
Samba QA Contact
http://bugs.debian.org/428618
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-11 13:37 UTC by Debian samba package maintainers (PUBLIC MAILING LIST)
Modified: 2009-05-18 13:09 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Debian samba package maintainers (PUBLIC MAILING LIST) 2008-08-11 13:37:14 UTC
Hello,

this is a forwarded bug from Debian (see http://bugs.debian.org/428618)

--8<--
smbd can fail to start if the "bind interfaces only" option has been
enabled, and "interfaces" is set to an interface that isn't enumerated
by samba due to a 128 MAX_INTERFACES limit.

    # rm -f /var/log/samba/log.smbd
    # smbd -d 10
    # grep interfaces /etc/samba/smb.conf 
       interfaces = 10.0.0.0/8
       bind interfaces only = true
    # grep "added interface" /var/log/samba/log.smbd | wc -l
    128
    # grep 10.0.0.0 /var/log/samba/log.smbd 
      doing parameter interfaces = 10.0.0.0/8
      Can't determine ip for broadcast address 10.0.0.0/8

In the full log, the 128 "added interface" messages are for IP addresses
on interface br0.  As br0 has more than 128 IP addresses, the IP
addresses on eth0 (the 10.0.0.0/8 network) aren't enumerated.
--8<--

source/include/interfaces.h:#define MAX_INTERFACES 128

This could be raised but there maybe a better solution for users with the need of many interfaces?

Thank you

noel@d.o
Comment 1 Volker Lendecke 2009-05-18 10:15:21 UTC
Checked in a fix to master with bbbf9f13add129. Jeremy, please review, merge for 3.4 and then close this bug. I don't think we should put this into 3.3.

Thanks,

Volker
Comment 2 Jeremy Allison 2009-05-18 13:09:12 UTC
Obvious goodness - thanks ! Merged to 3.4.
Jeremy.