fixed the problem that browsing breaks if you put quote marks around
authorAndrew Tridgell <tridge@samba.org>
Thu, 16 Oct 1997 03:53:07 +0000 (03:53 +0000)
committerAndrew Tridgell <tridge@samba.org>
Thu, 16 Oct 1997 03:53:07 +0000 (03:53 +0000)
commit69eea4fcf818ce976060e31432a6a415ab191d64
tree52f28a6fc81b4f88df5a1320372ddc3eb090715e
parentc3ceabce3afe7104d71199608807fa0fd8f41ce7
fixed the problem that browsing breaks if you put quote marks around
the server string option.

I fixed it by adding: trim_string(ret, "\"", "\"") to lp_string()
which means that it removes leading and trailing quotes from _all_
strings in smb.conf. I think this is what we want as I can't actually
think of any case where quoted strings are needed.

I suspect that this fix will actually fix browsing at quite a few
sites. It's not uncommon to see people putting "" marks around the
workgroup name or other essential strings.

The real reason it was causing browsing to fail is that the browse.dat
file uses " to delimit fields, which means that it wasn't being parsed
correctly. It would be nice to use a proper database format in
browse.dat sometime. In fact, we really need a generic database type
subsystem in Samba to replace the mish-mash we currrently use
(browse.dat, wins.dat, smbpasswd etc)
(This used to be commit ac50a88e1181bd4954f03e5450dedce87ed781ad)
source3/param/loadparm.c