mount: fix mount failure with old kernels

If the 'cmi->cmi_opts' has only one option and there is no any
comma in it, when removing the 'mon_addr' it will fail. And will
pass the 'mon_addr' option to the old kernels.

Fixes: https://tracker.ceph.com/issues/58018
Signed-off-by: Xiubo Li <xiubli@redhat.com>

diff --git a/src/mount/mount.ceph.c b/src/mount/mount.ceph.c
index 1aeac83dcc3459900dc6e9b3efabcf6385bb45e3..631b0b3fcf18de215bdd9ba9c71a40476adf29f4 100644 (file)
--- a/src/mount/mount.ceph.c
+++ b/src/mount/mount.ceph.c
@@ -166,7 +166,9 @@ static int remove_opt(struct ceph_mount_info *cmi, const char *key, char **value
                memmove(key_start, value_end, len2);
        } else {
                 /* last kv pair - swallow the comma */
-               --key_start;
+               if (*(key_start - 1) == ',') {
+                       --key_start;
+               }
                *key_start = '\0';
        }