Add comment explaining why io.in.workgroup isn't important.

This protocol feild isn't used by servers (apparently), so we might be
able to get rid of it.

Andrew Bartlett
(This used to be commit 58935acc7c8e97323d5d5979234ef26ef8a100a4)

diff --git a/source4/libcli/raw/clitree.c b/source4/libcli/raw/clitree.c
index d5075f9271dc23ebfb6216a91a729eb2555eeb8e..15cd70833c7c0b13810525ef7dfa4e632cde958a 100644 (file)
--- a/source4/libcli/raw/clitree.c
+++ b/source4/libcli/raw/clitree.c
@@ -193,6 +193,11 @@ NTSTATUS smbcli_tree_full_connection(TALLOC_CTX *parent_ctx,
        io.in.service_type = service_type;
        io.in.credentials = credentials;
        io.in.fallback_to_anonymous = false;
+
+       /* This workgroup gets sent out by the SPNEGO session setup.
+        * I don't know of any servers that look at it, so we might
+        * hardcode it to "" some day, when the war on global_loadparm
+        * is complete -- abartlet 2008-04-28 */
        io.in.workgroup = lp_workgroup(global_loadparm);
        io.in.options = *options;