Blob Blame History Raw
From 1c1e872ad43ab0860e05be679632097296eef174 Mon Sep 17 00:00:00 2001
From: Kaushal M <kaushal@redhat.com>
Date: Tue, 10 Jul 2018 20:56:08 +0530
Subject: [PATCH 686/689] glusterd: _is_prefix should handle 0-length paths

If one of the paths given to _is_prefix is 0-length, then it is not a
prefix of the other. Hence, _is_prefix should return false.

>Change-Id: I54aa577a64a58940ec91872d0d74dc19cff9106d
>fixes: bz#1599783
>Signed-off-by: Kaushal M <kaushal@redhat.com>

upstream patch: https://review.gluster.org/#/c/20490/

Change-Id: I54aa577a64a58940ec91872d0d74dc19cff9106d
BUG: 1599803
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
Reviewed-on: https://code.engineering.redhat.com/gerrit/143746
Tested-by: RHGS Build Bot <nigelb@redhat.com>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
---
 xlators/mgmt/glusterd/src/glusterd-utils.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c
index bbf6f7d..8e1048c 100644
--- a/xlators/mgmt/glusterd/src/glusterd-utils.c
+++ b/xlators/mgmt/glusterd/src/glusterd-utils.c
@@ -1269,6 +1269,15 @@ _is_prefix (char *str1, char *str2)
         len1 = strlen (str1);
         len2 = strlen (str2);
         small_len = min (len1, len2);
+
+        /*
+         * If either one (not both) of the strings are 0-length, they are not
+         * prefixes of each other.
+         */
+        if ((small_len == 0) && (len1 != len2)) {
+                return _gf_false;
+        }
+
         for (i = 0; i < small_len; i++) {
                 if (str1[i] != str2[i]) {
                         prefix = _gf_false;
-- 
1.8.3.1