On 1/22/20 10:46 AM, Ondrej Mosnacek wrote:
It is declared in an internal header but never defined. Remove it.
Signed-off-by: Ondrej Mosnacek <omosnace@xxxxxxxxxx>
---
libsepol/cil/src/cil_mem.h | 1 -
libsepol/cil/src/cil_strpool.c | 2 --
2 files changed, 3 deletions(-)
diff --git a/libsepol/cil/src/cil_mem.h b/libsepol/cil/src/cil_mem.h
index 902ce131..794f02a3 100644
--- a/libsepol/cil/src/cil_mem.h
+++ b/libsepol/cil/src/cil_mem.h
@@ -36,7 +36,6 @@ void *cil_calloc(size_t num_elements, size_t element_size);
void *cil_realloc(void *ptr, size_t size);
char *cil_strdup(const char *str);
int cil_asprintf(char **strp, const char *fmt, ...);
-void (*cil_mem_error_handler)(void);
#endif /* CIL_MEM_H_ */
diff --git a/libsepol/cil/src/cil_strpool.c b/libsepol/cil/src/cil_strpool.c
index 97d4c4b9..508541d6 100644
--- a/libsepol/cil/src/cil_strpool.c
+++ b/libsepol/cil/src/cil_strpool.c
@@ -80,7 +80,6 @@ char *cil_strpool_add(const char *str)
int rc = hashtab_insert(cil_strpool_tab, (hashtab_key_t)strpool_ref->str, strpool_ref);
if (rc != SEPOL_OK) {
pthread_mutex_unlock(&cil_strpool_mutex);
- (*cil_mem_error_handler)();
pthread_mutex_lock(&cil_strpool_mutex);
}
}
I removed the cil_mem_error_handler stuff last Fall (see commit 4459d635). I
guess that I must have missed these somehow.
In the other places where I removed it, I replaced it with the inlined default
hanlder which consisted of the following:
cil_log(CIL_ERR, "Failed to allocate memory\n");
exit(1);
Since we are existing, the pthread_mutex_lock() call can be removed.
@@ -104,7 +103,6 @@ void cil_strpool_init(void)
cil_strpool_tab = hashtab_create(cil_strpool_hash, cil_strpool_compare, CIL_STRPOOL_TABLE_SIZE);
if (cil_strpool_tab == NULL) {
pthread_mutex_unlock(&cil_strpool_mutex);
- (*cil_mem_error_handler)();
return;
}
}
Same thing here, but now the return won't be needed.
Thanks,
Jim
--
James Carter <jwcart2@xxxxxxxxxxxxx>
National Security Agency