|
 |
d1681e |
From f62b21ca4a6e745f8589b257574aa2304e43871b Mon Sep 17 00:00:00 2001
|
|
|
d1681e |
From: Kaushal M <kaushal@redhat.com>
|
|
|
d1681e |
Date: Tue, 10 Jul 2018 20:56:08 +0530
|
|
|
d1681e |
Subject: [PATCH 319/325] glusterd: _is_prefix should handle 0-length paths
|
|
|
d1681e |
|
|
|
d1681e |
If one of the paths given to _is_prefix is 0-length, then it is not a
|
|
|
d1681e |
prefix of the other. Hence, _is_prefix should return false.
|
|
|
d1681e |
|
|
|
d1681e |
>Change-Id: I54aa577a64a58940ec91872d0d74dc19cff9106d
|
|
|
d1681e |
>fixes: bz#1599783
|
|
|
d1681e |
>Signed-off-by: Kaushal M <kaushal@redhat.com>
|
|
|
d1681e |
|
|
|
d1681e |
upstream patch:https://review.gluster.org/#/c/20490/
|
|
|
d1681e |
|
|
|
d1681e |
Change-Id: I54aa577a64a58940ec91872d0d74dc19cff9106d
|
|
|
d1681e |
BUG: 1599823
|
|
|
d1681e |
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
|
|
|
d1681e |
Reviewed-on: https://code.engineering.redhat.com/gerrit/143743
|
|
|
d1681e |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
d1681e |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
d1681e |
---
|
|
|
d1681e |
xlators/mgmt/glusterd/src/glusterd-utils.c | 9 +++++++++
|
|
|
d1681e |
1 file changed, 9 insertions(+)
|
|
|
d1681e |
|
|
|
d1681e |
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c
|
|
|
d1681e |
index f62c917..565b6c5 100644
|
|
|
d1681e |
--- a/xlators/mgmt/glusterd/src/glusterd-utils.c
|
|
|
d1681e |
+++ b/xlators/mgmt/glusterd/src/glusterd-utils.c
|
|
|
d1681e |
@@ -1318,6 +1318,15 @@ _is_prefix (char *str1, char *str2)
|
|
|
d1681e |
len1 = strlen (str1);
|
|
|
d1681e |
len2 = strlen (str2);
|
|
|
d1681e |
small_len = min (len1, len2);
|
|
|
d1681e |
+
|
|
|
d1681e |
+ /*
|
|
|
d1681e |
+ * If either one (not both) of the strings are 0-length, they are not
|
|
|
d1681e |
+ * prefixes of each other.
|
|
|
d1681e |
+ */
|
|
|
d1681e |
+ if ((small_len == 0) && (len1 != len2)) {
|
|
|
d1681e |
+ return _gf_false;
|
|
|
d1681e |
+ }
|
|
|
d1681e |
+
|
|
|
d1681e |
for (i = 0; i < small_len; i++) {
|
|
|
d1681e |
if (str1[i] != str2[i]) {
|
|
|
d1681e |
prefix = _gf_false;
|
|
|
d1681e |
--
|
|
|
d1681e |
1.8.3.1
|
|
|
d1681e |
|