If a user specified a member that isn't a LIST, a USER, or a foreign

string and didn't explicitly specify STRING:, they probably typoed.

This is old behavior that broke; restore it.

diff --git a/clients/blanche/blanche.c b/clients/blanche/blanche.c
index ed79223e6bc67a6ac8515fa4f2b8d3e66d3194e6..64782396f4efd2f1566b657b1dafb5272af58d61 100644 (file)
--- a/clients/blanche/blanche.c
+++ b/clients/blanche/blanche.c
@@ -660,13 +660,7 @@ int main(int argc, char **argv)
            }
        case M_STRING:
          status = mrcl_validate_string_member(memberstruct->name);
-         mrcl_com_err(whoami);
-         if (status == MRCL_REJECT)
-           {
-             success = 0;
-             break;
-           }
-         else if (memberstruct->type == M_ANY && status != MR_SUCCESS)
+         if (memberstruct->type == M_ANY && status == MRCL_WARN)
            {
              /* if user is trying to add something which isn't a
                 remote string, or a list, or a user, and didn't
@@ -676,6 +670,14 @@ int main(int argc, char **argv)
              success = 0;
              break;
            }
+         else
+           mrcl_com_err(whoami);
+
+         if (status == MRCL_REJECT)
+           {
+             success = 0;
+             break;
+           }
 
          membervec[1] = "STRING";
          status = mr_query("add_tagged_member_to_list", 4, membervec,