if we are truncating to the . we need to start at the beginning in case

there are multiple "."'s in the name.

This code is protected with an #ifdef TRUNCATE_NETBIOS_NAME and this
is #define'd to 1 directly above. Should we also get rid of the #ifdef?
(This used to be commit 0375dace248eb3dc660fa2bca2808552e502b3f7)

diff --git a/source3/libsmb/nmblib.c b/source3/libsmb/nmblib.c
index ff38245435dcbb09753488d6a97a7cd3a46c179b..d0ea57dc1ce9e3daa21fbface35090b0d6989831 100644 (file)
--- a/source3/libsmb/nmblib.c
+++ b/source3/libsmb/nmblib.c
@@ -1116,7 +1116,7 @@ char *dns_to_netbios_name(char *dns_name)
           netbios name up to and including the '.'  this even applies, by
           mistake, to workgroup (domain) names, which is _really_ daft.
         */
-       for (i = 15; i >= 0; i--)
+       for (i = 0; i >= 15; i--)
        {
                if (netbios_name[i] == '.')
                {