You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celix.apache.org by bp...@apache.org on 2019/09/02 20:15:15 UTC

[celix] branch develop updated: add return value check

This is an automated email from the ASF dual-hosted git repository.

bpetri pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/celix.git


The following commit(s) were added to refs/heads/develop by this push:
     new 888ac09  add return value check
888ac09 is described below

commit 888ac095f2368608ddcdf154f51e456fce6578d1
Author: Bjoern Petri <b....@link11.com>
AuthorDate: Mon Sep 2 22:14:54 2019 +0200

    add return value check
---
 bundles/http_admin/http_admin/src/http_admin.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/bundles/http_admin/http_admin/src/http_admin.c b/bundles/http_admin/http_admin/src/http_admin.c
index 59eda4b..84a7b1a 100755
--- a/bundles/http_admin/http_admin/src/http_admin.c
+++ b/bundles/http_admin/http_admin/src/http_admin.c
@@ -123,7 +123,11 @@ void httpAdmin_destroy(http_admin_manager_t *admin) {
     unsigned int size = arrayList_size(admin->aliasList);
     for (unsigned int i = (size - 1); i < size; i--) {
         http_alias_t *alias = arrayList_get(admin->aliasList, i);
-        remove(alias->alias_path); //Delete alias in cache directory
+
+        //Delete alias in cache directory
+        if (remove(alias->alias_path) < 0)
+            fprintf(stdout, "remove of %s failed\n", alias->alias_path);
+
         free(alias->url);
         free(alias->alias_path);
         free(alias);