Re: [Qemu-devel] SAMBA support in QEMU

[Fabrice Bellard <fabrice@bellard.org>]

OK. 'smb ports=0' seems a good idea. Can you find why there is still a 
datagram connection opened ?

I still wonder if the best solution would be to ship a specific smbd 
version with QEMU...

Fabrice.

Bernhard Huafbauer wrote:
> Hello,
> I found nothing about disabling the opening of the ports
> in the man page of the smb.conf.
> The only thing I have found to prevent smbd to open a port is 
> to give a config file with one of these lines:
> 
> 	smb ports=" "
> 	smb ports=' '
> 	smb ports=0
> 
> the samba source file /source/smbd/server.c line 242 (version 3.0.2a-SUSE) look like this:
> ...
> 			for (ptr=ports; next_token(&ptr, tok, NULL, sizeof(tok)); ) {
> 				unsigned port = atoi(tok);
> 				if (port == 0) continue;
> 				s = fd_listenset[num_sockets] = open_socket_in(SOCK_STREAM, port, 0, ifip->s_addr, True);
> ...
> ...
> 		for (ptr=ports; next_token(&ptr, tok, NULL, sizeof(tok)); ) {
> 			unsigned port = atoi(tok);
> 			if (port == 0) continue;
> 			/* open an incoming socket */
> 			s = open_socket_in(SOCK_STREAM, port, 0,
> ...
> 
> Interesting the comparison to 0 and the continue ...
> so setting it to 0 would be best the way?
> 
> with this setting a smbd called by hand doesn't open any port except this DGRAM thing
> 	bernhard@wombl:/tmp/qemu-smb.8091> ps aux | grep smbd | grep -v grep
> 	bernhard 10087  0.0  0.5  9912 2800 ?        Ss   21:16   0:00 /usr/sbin/smbd -s /tmp/qemu-smb.8091/smb.conf
> 
> 	bernhard@wombl:/tmp/qemu-smb.8091> netstat -anp | grep 10087
> 	unix  2      [ ]         DGRAM                    23644  10087/smbd
> 
> when running qemu with this setting and accessing \\10.0.2.4\qemu
> 	bernhard@wombl:/tmp> ps aux | grep smbd
> 	bernhard 11976  0.0  0.0     0    0 ?        Z    22:02   0:00 [smbd] <defunct>
> 	bernhard 12012  0.0  0.0     0    0 ?        Z    22:03   0:00 [smbd] <defunct>
> 	bernhard 12045  0.1  0.6 10984 3492 ?        S    22:03   0:00 /usr/sbin/smbd -s /tmp/qemu-smb.11876/smb.conf
> 	bernhard 12064  0.0  0.1  2648  736 pts/1    S+   22:04   0:00 grep smbd
> 
> and sometimes somthing like this (?):
> 	bernhard@wombl:/tmp> netstat -anp | grep 12045
> 	tcp        0      0 127.0.0.1:1357          127.0.0.1:1356          VERBUNDEN   12045/smbd
> 	udp        0      0 127.0.0.1:1156          0.0.0.0:*                           12045/smbd
> 
> 
> (As a sidenote:
> the smb_exit cleans not all of the temp directory
> there is still a directory printing so the last rmdir
> call fails and a /tmp/qemu-smb.10542/printing/ remains.)
> 
> Regards
> Bernhard
> 
> 
> Am Montag, 20. Dezember 2004 00:40 schrieb Fabrice Bellard:
> 
>>Hi,
>>
>>It would be cleaner to find a way to disable the access to the port 445 
>>(it seems to be an uneeded feature for QEMU and your patch may not work 
>>if several instances of QEMU are launched). Can you look at the SAMBA 
>>documentation to see if there is an option to do that ?
>>
>>Fabrice.
>>
>>Bernhard Huafbauer wrote:
>>
>>>Hello,
>>>
>>>With this little changes to the samba configuration
>>>file I was able to access the share from the guest os
>>>(winxp-oem-german-nosp, samba 3.0.2a-SUSE)
>>>
>>>Don't know how it would affect some samba 2.x installations ...
>>>
>>>The entry smb ports is because smbd wants to open port 445
>>>(an I'm not allowed to as user).
>>>
>>>Regards,
>>>Bernhard
>>>
>>>Am Donnerstag, 9. Dezember 2004 22:34 schrieb Adrian Smarzewski:
>>>
>>>
>>>>David Still wrote:
>>>>
>>>>
>>>>>This could also be why SAMBA support does not appear to work on Mac OS 
>>>>>X.  Version 10.3.6  appears to use SAMBA version 3.0.5.
>>>>
>>>>3.0 was released more than year ago I think...
>>>>Maybe It's more important to support 3.0.x than 2.x now.
>>>>
>>>
>>>
>>>bernhard@wombl:~/projekte/software/qemu> diff -Nru orig/qemu-cvs/qemu/vl.c qemu-cvs/qemu/vl.c
>>>--- orig/qemu-cvs/qemu/vl.c     2004-12-17 21:10:34.000000000 +0100
>>>+++ qemu-cvs/qemu/vl.c  2004-12-19 15:44:27.501826076 +0100
>>>@@ -1535,6 +1535,9 @@
>>>     }
>>>     fprintf(f,
>>>             "[global]\n"
>>>+            "private dir=%s\n"
>>>+            "smb ports=2445 2139\n"
>>>+            "socket address=127.0.0.1\n"
>>>             "pid directory=%s\n"
>>>             "lock directory=%s\n"
>>>             "log file=%s/log.smbd\n"
>>>@@ -1548,6 +1551,7 @@
>>>             smb_dir,
>>>             smb_dir,
>>>             smb_dir,
>>>+            smb_dir,
>>>             exported_dir
>>>             );
>>>     fclose(f);
>>>
>>>
>>>_______________________________________________
>>>Qemu-devel mailing list
>>>Qemu-devel@nongnu.org
>>>http://lists.nongnu.org/mailman/listinfo/qemu-devel
>>>
>>>
>>>
>>
>>
>>
>>_______________________________________________
>>Qemu-devel mailing list
>>Qemu-devel@nongnu.org
>>http://lists.nongnu.org/mailman/listinfo/qemu-devel
>>
> 
> 
> 
> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel
> 
> 
>