|
|
1756dc |
From 67b5d7778cad682bf8046973900caf2fa3353d0e Mon Sep 17 00:00:00 2001
|
|
|
1756dc |
From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <pbrezina@redhat.com>
|
|
|
1756dc |
Date: Wed, 28 Nov 2018 13:59:51 +0100
|
|
|
1756dc |
Subject: [PATCH 14/15] util: do not return value from string_array_del_value
|
|
|
1756dc |
|
|
|
1756dc |
It is not needed.
|
|
|
1756dc |
---
|
|
|
1756dc |
src/lib/authselect.c | 9 ++-------
|
|
|
1756dc |
src/lib/util/string_array.c | 6 +++---
|
|
|
1756dc |
src/lib/util/string_array.h | 2 +-
|
|
|
1756dc |
src/tests/test_util_string_array.c | 28 ++++++++--------------------
|
|
|
1756dc |
4 files changed, 14 insertions(+), 31 deletions(-)
|
|
|
1756dc |
|
|
|
1756dc |
diff --git a/src/lib/authselect.c b/src/lib/authselect.c
|
|
|
1756dc |
index e0b8b1246b0e7139494d90cca4e0ebed3eb66376..0f8d4a8b6d0b0faef81daf176486108ed0ea74db 100644
|
|
|
1756dc |
--- a/src/lib/authselect.c
|
|
|
1756dc |
+++ b/src/lib/authselect.c
|
|
|
1756dc |
@@ -179,7 +179,7 @@ authselect_apply_changes(void)
|
|
|
1756dc |
WARN("Profile feature [%s] is no longer supported, removing it...",
|
|
|
1756dc |
features[i]);
|
|
|
1756dc |
|
|
|
1756dc |
- features = string_array_del_value(features, features[i]);
|
|
|
1756dc |
+ string_array_del_value(features, features[i]);
|
|
|
1756dc |
i--;
|
|
|
1756dc |
}
|
|
|
1756dc |
|
|
|
1756dc |
@@ -247,15 +247,10 @@ authselect_feature_disable(const char *feature)
|
|
|
1756dc |
return ret;
|
|
|
1756dc |
}
|
|
|
1756dc |
|
|
|
1756dc |
- features = string_array_del_value(features, feature);
|
|
|
1756dc |
- if (features == NULL) {
|
|
|
1756dc |
- ret = ENOMEM;
|
|
|
1756dc |
- goto done;
|
|
|
1756dc |
- }
|
|
|
1756dc |
+ string_array_del_value(features, feature);
|
|
|
1756dc |
|
|
|
1756dc |
ret = authselect_activate(profile_id, (const char **)features, false);
|
|
|
1756dc |
|
|
|
1756dc |
-done:
|
|
|
1756dc |
string_array_free(features);
|
|
|
1756dc |
free(profile_id);
|
|
|
1756dc |
|
|
|
1756dc |
diff --git a/src/lib/util/string_array.c b/src/lib/util/string_array.c
|
|
|
1756dc |
index a8afa5ab8edbb26d6f946619f9ce0b83c511bb8c..e8871dc067fbf3d461d1ee9579813ddc81eef676 100644
|
|
|
1756dc |
--- a/src/lib/util/string_array.c
|
|
|
1756dc |
+++ b/src/lib/util/string_array.c
|
|
|
1756dc |
@@ -137,7 +137,7 @@ string_array_add_value(char **array, const char *value, bool unique)
|
|
|
1756dc |
return string_array_add_value_safe(array, value, strlen(value), unique);
|
|
|
1756dc |
}
|
|
|
1756dc |
|
|
|
1756dc |
-char **
|
|
|
1756dc |
+void
|
|
|
1756dc |
string_array_del_value(char **array, const char *value)
|
|
|
1756dc |
{
|
|
|
1756dc |
size_t count;
|
|
|
1756dc |
@@ -145,7 +145,7 @@ string_array_del_value(char **array, const char *value)
|
|
|
1756dc |
size_t i;
|
|
|
1756dc |
|
|
|
1756dc |
if (array == NULL) {
|
|
|
1756dc |
- return NULL;
|
|
|
1756dc |
+ return;
|
|
|
1756dc |
}
|
|
|
1756dc |
|
|
|
1756dc |
count = string_array_count(array);
|
|
|
1756dc |
@@ -167,7 +167,7 @@ string_array_del_value(char **array, const char *value)
|
|
|
1756dc |
array[pos] = NULL;
|
|
|
1756dc |
}
|
|
|
1756dc |
|
|
|
1756dc |
- return array;
|
|
|
1756dc |
+ return;
|
|
|
1756dc |
}
|
|
|
1756dc |
|
|
|
1756dc |
char **
|
|
|
1756dc |
diff --git a/src/lib/util/string_array.h b/src/lib/util/string_array.h
|
|
|
1756dc |
index ba9760b5d66a9619ca8edea5e3418c5cfbbec929..5842db174563982528e20354138ef5792346fb37 100644
|
|
|
1756dc |
--- a/src/lib/util/string_array.h
|
|
|
1756dc |
+++ b/src/lib/util/string_array.h
|
|
|
1756dc |
@@ -115,7 +115,7 @@ string_array_add_value(char **array, const char *value, bool unique);
|
|
|
1756dc |
*
|
|
|
1756dc |
* @return Array without the value.
|
|
|
1756dc |
*/
|
|
|
1756dc |
-char **
|
|
|
1756dc |
+void
|
|
|
1756dc |
string_array_del_value(char **array, const char *value);
|
|
|
1756dc |
|
|
|
1756dc |
/**
|
|
|
1756dc |
diff --git a/src/tests/test_util_string_array.c b/src/tests/test_util_string_array.c
|
|
|
1756dc |
index 249cb96acea3c4feac910702572cafb1025d9496..ad76f8b190b823210b5e30ae828dce6518596e3b 100644
|
|
|
1756dc |
--- a/src/tests/test_util_string_array.c
|
|
|
1756dc |
+++ b/src/tests/test_util_string_array.c
|
|
|
1756dc |
@@ -51,8 +51,7 @@ void test_string_array_del_value__single(void **state)
|
|
|
1756dc |
assert_null(array[i]);
|
|
|
1756dc |
|
|
|
1756dc |
/* Delete value. */
|
|
|
1756dc |
- array = string_array_del_value(array, "2");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
+ string_array_del_value(array, "2");
|
|
|
1756dc |
|
|
|
1756dc |
/* Test values. */
|
|
|
1756dc |
for (i = 0; expected[i] != NULL; i++) {
|
|
|
1756dc |
@@ -83,8 +82,7 @@ void test_string_array_del_value__single_repeated(void **state)
|
|
|
1756dc |
assert_null(array[i]);
|
|
|
1756dc |
|
|
|
1756dc |
/* Delete value. */
|
|
|
1756dc |
- array = string_array_del_value(array, "2");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
+ string_array_del_value(array, "2");
|
|
|
1756dc |
|
|
|
1756dc |
/* Test values. */
|
|
|
1756dc |
for (i = 0; expected[i] != NULL; i++) {
|
|
|
1756dc |
@@ -115,14 +113,9 @@ void test_string_array_del_value__multiple(void **state)
|
|
|
1756dc |
assert_null(array[i]);
|
|
|
1756dc |
|
|
|
1756dc |
/* Delete value. */
|
|
|
1756dc |
- array = string_array_del_value(array, "2");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
-
|
|
|
1756dc |
- array = string_array_del_value(array, "3");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
-
|
|
|
1756dc |
- array = string_array_del_value(array, "5");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
+ string_array_del_value(array, "2");
|
|
|
1756dc |
+ string_array_del_value(array, "3");
|
|
|
1756dc |
+ string_array_del_value(array, "5");
|
|
|
1756dc |
|
|
|
1756dc |
/* Test values. */
|
|
|
1756dc |
for (i = 0; expected[i] != NULL; i++) {
|
|
|
1756dc |
@@ -153,14 +146,9 @@ void test_string_array_del_value__multiple_repeated(void **state)
|
|
|
1756dc |
assert_null(array[i]);
|
|
|
1756dc |
|
|
|
1756dc |
/* Delete value. */
|
|
|
1756dc |
- array = string_array_del_value(array, "2");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
-
|
|
|
1756dc |
- array = string_array_del_value(array, "3");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
-
|
|
|
1756dc |
- array = string_array_del_value(array, "5");
|
|
|
1756dc |
- assert_non_null(array);
|
|
|
1756dc |
+ string_array_del_value(array, "2");
|
|
|
1756dc |
+ string_array_del_value(array, "3");
|
|
|
1756dc |
+ string_array_del_value(array, "5");
|
|
|
1756dc |
|
|
|
1756dc |
/* Test values. */
|
|
|
1756dc |
for (i = 0; expected[i] != NULL; i++) {
|
|
|
1756dc |
--
|
|
|
1756dc |
2.17.2
|
|
|
1756dc |
|