> > During examining BUG#221, I found a bug?
> > At nmbd_incomingdgrams.c:111 there is a line
> > comment = 0;
> > This makes browselist comment truncated 12 bytes.
> > For example, "Samba CVS 3.0.0rc2" becomes "Samba CVS 3.".
> > Why was this change committed?
Samba 3.0.0rc1 does not have this bug, also Samba 2.2.3 on Debian works well.
This is not a severe, but a bug.
On Fri, Aug 29, 2003 at 05:24:35PM +0000, Jeremy Allison wrote:
> My mistake. I was reading the original code when fixing
> nmbd w.r.t. mb character set issues and wanted to ensure
> I didn't change the behaviour. I'll check in the 2.2
> codebase but it looked like the same was done there. If
> not please log a bug and I'll fix it asap.
Hmmm... Share names are limited to 12 bytes in RAP, though the RAP data
descriptor string lists 13 bytes to account for a nul terminator. Could
there be some confusion between the share name and comment fields?
Just a wild guess...
It sounds like this is fixed in CVS now:
Date: Mon Sep 1 18:28:09 2003
Update of /data/cvs/samba/source/nmbd
In directory dp.samba.org:/tmp/cvs-serv6008/nmbd
Fix comment strings to 43 bytes as per spec.
nmbd_incomingdgrams.c 126.96.36.199 => 188.8.131.52
originally reported against one of the 3.0.0rc[1-4] releases.
Cleaning up non-production versions.
sorry for the same, cleaning up the database to prevent unecessary reopens of bugs.