Hello all,
when testing samba 2.0.7 with service names containing umlauts
we noticed duplicate entries of shares under samba server machine. Not
all of services were duplicated but just those containing umlauts.
The problematic function is add_a_service() in module param:loadparm.c.
The problem is when creating service structure, name is transformad
with unix_to_dos(). But, when searching if service already exists, the
original name is used and therefore service is never found. The problem
appears after SIGHUP when services get reloaded.
Attached you can find patch and cvs log message.
Regards,
Tine.
-------------- next part --------------
2000-06-19 10:05 Tine Smukavec <tine.smukavec@hermes.si>
* param/loadparm.c (add_a_service): transform incoming service name
before searching for existing service.
-------------- next part --------------
Index: loadparm.c
==================================================================RCS file:
/cvsroot/samba/source/param/loadparm.c,v
retrieving revision 1.170.2.59
diff -u -r1.170.2.59 loadparm.c
--- loadparm.c 2000/05/17 01:12:25 1.170.2.59
+++ loadparm.c 2000/06/19 07:46:38
@@ -1542,7 +1542,13 @@
/* it might already exist */
if (name)
{
- i = getservicebyname(name,NULL);
+ char *service_loc = NULL;
+
+ string_set(&service_loc,name);
+ unix_to_dos(service_loc, True);
+ i = getservicebyname(service_loc,NULL);
+ string_free(&service_loc);
+
if (i >= 0)
return(i);
}