Commit 3a6b9337 authored by Shawn Webb's avatar Shawn Webb
Browse files

Don't expose functions that should be static



Even the function declarations were gated behind _lattutil_internal, the
functions can be declared static in their respective files.
Signed-off-by: Shawn Webb's avatarShawn Webb <shawn.webb@hardenedbsd.org>
parent 3fbd9a1e
......@@ -514,45 +514,6 @@ const char *lattutil_sqlite_get_column_string(const ucl_object_t *, size_t);
int64_t lattutil_sqlite_get_column_int(const ucl_object_t *, size_t, int64_t);
#ifdef _lattutil_internal
ssize_t lattutil_log_syslog_debug(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_syslog_err(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_syslog_info(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_syslog_warn(lattutil_log_t *, int,
const char *, ...);
void lattutil_log_syslog_close(lattutil_log_t *);
ssize_t lattutil_log_dummy_debug(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_dummy_err(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_dummy_info(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_dummy_warn(lattutil_log_t *, int,
const char *, ...);
void lattutil_log_dummy_close(lattutil_log_t *);
ssize_t lattutil_log_stdio_debug(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_stdio_err(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_stdio_info(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_stdio_warn(lattutil_log_t *, int,
const char *, ...);
void lattutil_log_stdio_close(lattutil_log_t *);
ssize_t lattutil_log_sqlite_debug(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_sqlite_err(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_sqlite_info(lattutil_log_t *, int,
const char *, ...);
ssize_t lattutil_log_sqlite_warn(lattutil_log_t *, int,
const char *, ...);
void lattutil_log_sqlite_close(lattutil_log_t *);
#endif /* _lattutil_internal */
#ifdef __cplusplus
......
......@@ -29,52 +29,62 @@
#include "liblattutil.h"
static ssize_t _lattutil_log_dummy_debug(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_dummy_err(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_dummy_info(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_dummy_warn(lattutil_log_t *, int,
const char *, ...);
static void _lattutil_log_dummy_close(lattutil_log_t *);
EXPORTED_SYM
void
lattutil_log_dummy_init(lattutil_log_t *res)
{
res->ll_log_close = lattutil_log_dummy_close;
res->ll_log_debug = lattutil_log_dummy_debug;
res->ll_log_err = lattutil_log_dummy_err;
res->ll_log_info = lattutil_log_dummy_info;
res->ll_log_warn = lattutil_log_dummy_warn;
res->ll_log_close = _lattutil_log_dummy_close;
res->ll_log_debug = _lattutil_log_dummy_debug;
res->ll_log_err = _lattutil_log_dummy_err;
res->ll_log_info = _lattutil_log_dummy_info;
res->ll_log_warn = _lattutil_log_dummy_warn;
}
ssize_t
lattutil_log_dummy_debug(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_dummy_debug(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
return (0);
}
ssize_t
lattutil_log_dummy_err(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_dummy_err(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
return (0);
}
ssize_t
lattutil_log_dummy_info(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_dummy_info(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
return (0);
}
ssize_t
lattutil_log_dummy_warn(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_dummy_warn(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
return (0);
}
void
lattutil_log_dummy_close(lattutil_log_t *logp)
static void
_lattutil_log_dummy_close(lattutil_log_t *logp)
{
return;
......
......@@ -36,6 +36,18 @@ typedef struct _log_sqlite3_aux {
char *lsc_table;
} log_sqlite3_ctx_t;
static ssize_t _lattutil_log_sqlite_debug(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_sqlite_err(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_sqlite_info(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_sqlite_warn(lattutil_log_t *, int,
const char *, ...);
static void _lattutil_log_sqlite_close(lattutil_log_t *);
static bool _lattutil_log_sqlite_dolog(log_sqlite3_ctx_t *,
const char *, const char *);
EXPORTED_SYM
bool
lattutil_log_sqlite_init(lattutil_log_t *logp,
......@@ -85,11 +97,11 @@ lattutil_log_sqlite_init(lattutil_log_t *logp,
return (false);
}
logp->ll_log_close = lattutil_log_sqlite_close;
logp->ll_log_debug = lattutil_log_sqlite_debug;
logp->ll_log_err = lattutil_log_sqlite_err;
logp->ll_log_info = lattutil_log_sqlite_info;
logp->ll_log_warn = lattutil_log_sqlite_warn;
logp->ll_log_close = _lattutil_log_sqlite_close;
logp->ll_log_debug = _lattutil_log_sqlite_debug;
logp->ll_log_err = _lattutil_log_sqlite_err;
logp->ll_log_info = _lattutil_log_sqlite_info;
logp->ll_log_warn = _lattutil_log_sqlite_warn;
lattutil_log_set_aux(logp, ctx, sizeof(*ctx));
......@@ -99,28 +111,20 @@ lattutil_log_sqlite_init(lattutil_log_t *logp,
return (true);
}
ssize_t
lattutil_log_sqlite_debug(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_sqlite_debug(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
lattutil_sqlite_query_t *query;
log_sqlite3_ctx_t *ctx;
char *msg, *querystr;
struct timeval tv;
va_list args;
size_t len;
char *msg;
if (logp == NULL || fmt == NULL) {
return (-1);
}
memset(&tv, 0, sizeof(tv));
gettimeofday(&tv, NULL);
msg = NULL;
query = NULL;
querystr = NULL;
va_start(args, fmt);
if (verbose == -1 || verbose >= logp->ll_verbosity) {
len = 0;
......@@ -137,75 +141,32 @@ lattutil_log_sqlite_debug(lattutil_log_t *logp, int verbose,
}
len = strlen(msg);
querystr = NULL;
asprintf(&querystr, "INSERT INTO %s ("
"timesec, timeusec, msgtype, msg) VALUES (?, ?, ?, ?)",
ctx->lsc_table);
if (querystr == NULL) {
len = -1;
goto end;
}
query = lattutil_sqlite_prepare(ctx->lsc_sqlctx, querystr);
if (query == NULL) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 1, tv.tv_sec) == false) {
if (_lattutil_log_sqlite_dolog(ctx, "DEBUG", msg)
== false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 2, tv.tv_usec) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 3, "DEBUG") == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 4, msg) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_exec(query) == false) {
len = -1;
goto end;
}
}
va_end(args);
end:
free(msg);
lattutil_sqlite_query_free(&query);
free(querystr);
return (len);
}
ssize_t
lattutil_log_sqlite_err(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_sqlite_err(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
lattutil_sqlite_query_t *query;
log_sqlite3_ctx_t *ctx;
char *msg, *querystr;
struct timeval tv;
va_list args;
size_t len;
char *msg;
if (logp == NULL || fmt == NULL) {
return (-1);
}
memset(&tv, 0, sizeof(tv));
gettimeofday(&tv, NULL);
msg = NULL;
query = NULL;
querystr = NULL;
va_start(args, fmt);
if (verbose == -1 || verbose >= logp->ll_verbosity) {
len = 0;
......@@ -222,73 +183,32 @@ lattutil_log_sqlite_err(lattutil_log_t *logp, int verbose,
}
len = strlen(msg);
querystr = NULL;
asprintf(&querystr, "INSERT INTO %s ("
"timesec, timeusec, msgtype, msg) VALUES (?, ?, ?, ?)",
ctx->lsc_table);
if (querystr == NULL) {
len = -1;
goto end;
}
query = lattutil_sqlite_prepare(ctx->lsc_sqlctx, querystr);
if (query == NULL) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 1, tv.tv_sec) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 2, tv.tv_usec) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 3, "ERROR") == false) {
if (_lattutil_log_sqlite_dolog(ctx, "ERROR", msg)
== false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 4, msg) == false) {
len = -1;
goto end;
}
lattutil_sqlite_exec(query);
}
va_end(args);
end:
lattutil_sqlite_query_free(&query);
free(msg);
free(querystr);
return (len);
}
ssize_t
lattutil_log_sqlite_info(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_sqlite_info(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
lattutil_sqlite_query_t *query;
log_sqlite3_ctx_t *ctx;
char *msg, *querystr;
struct timeval tv;
va_list args;
size_t len;
char *msg;
if (logp == NULL || fmt == NULL) {
return (-1);
}
memset(&tv, 0, sizeof(tv));
gettimeofday(&tv, NULL);
msg = NULL;
query = NULL;
querystr = NULL;
va_start(args, fmt);
if (verbose == -1 || verbose >= logp->ll_verbosity) {
len = 0;
......@@ -305,73 +225,32 @@ lattutil_log_sqlite_info(lattutil_log_t *logp, int verbose,
}
len = strlen(msg);
querystr = NULL;
asprintf(&querystr, "INSERT INTO %s ("
"timesec, timeusec, msgtype, msg) VALUES (?, ?, ?, ?)",
ctx->lsc_table);
if (querystr == NULL) {
len = -1;
goto end;
}
query = lattutil_sqlite_prepare(ctx->lsc_sqlctx, querystr);
if (query == NULL) {
if (_lattutil_log_sqlite_dolog(ctx, "INFO", msg)
== false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 1, tv.tv_sec) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 2, tv.tv_usec) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 3, "INFO") == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 4, msg) == false) {
len = -1;
goto end;
}
lattutil_sqlite_exec(query);
}
va_end(args);
end:
lattutil_sqlite_query_free(&query);
free(msg);
free(querystr);
return (len);
}
ssize_t
lattutil_log_sqlite_warn(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_sqlite_warn(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
lattutil_sqlite_query_t *query;
log_sqlite3_ctx_t *ctx;
char *msg, *querystr;
struct timeval tv;
va_list args;
size_t len;
char *msg;
if (logp == NULL || fmt == NULL) {
return (-1);
}
memset(&tv, 0, sizeof(tv));
gettimeofday(&tv, NULL);
msg = NULL;
query = NULL;
querystr = NULL;
va_start(args, fmt);
if (verbose == -1 || verbose >= logp->ll_verbosity) {
len = 0;
......@@ -388,54 +267,20 @@ lattutil_log_sqlite_warn(lattutil_log_t *logp, int verbose,
}
len = strlen(msg);
querystr = NULL;
asprintf(&querystr, "INSERT INTO %s ("
"timesec, timeusec, msgtype, msg) VALUES (?, ?, ?, ?)",
ctx->lsc_table);
if (querystr == NULL) {
if (_lattutil_log_sqlite_dolog(ctx, "WARNING", msg)
== false) {
len = -1;
goto end;
}
query = lattutil_sqlite_prepare(ctx->lsc_sqlctx, querystr);
if (query == NULL) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 1, tv.tv_sec) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_int(query, 2, tv.tv_usec) == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 3, "WARNING") == false) {
len = -1;
goto end;
}
if (lattutil_sqlite_bind_string(query, 4, msg) == false) {
len = -1;
goto end;
}
lattutil_sqlite_exec(query);
}
va_end(args);
end:
lattutil_sqlite_query_free(&query);
free(msg);
free(querystr);
return (len);
}
void
lattutil_log_sqlite_close(lattutil_log_t *logp)
static void
_lattutil_log_sqlite_close(lattutil_log_t *logp)
{
log_sqlite3_ctx_t *ctx;
size_t sz;
......@@ -452,3 +297,58 @@ lattutil_log_sqlite_close(lattutil_log_t *logp)
free(ctx->lsc_table);
}
static bool
_lattutil_log_sqlite_dolog(log_sqlite3_ctx_t *ctx, const char *msgtype, const char *msg)
{
lattutil_sqlite_query_t *query;
struct timeval tv;
char *querystr;
bool res;
memset(&tv, 0, sizeof(tv));
gettimeofday(&tv, NULL);
res = true;
query = NULL;
querystr = NULL;
asprintf(&querystr, "INSERT INTO %s ("
"timesec, timeusec, msgtype, msg) VALUES (?, ?, ?, ?)",
ctx->lsc_table);
if (querystr == NULL) {
res = false;
goto end;
}
query = lattutil_sqlite_prepare(ctx->lsc_sqlctx, querystr);
if (query == NULL) {
res = false;
goto end;
}
if (lattutil_sqlite_bind_int(query, 1, tv.tv_sec) == false) {
res = false;
goto end;
}
if (lattutil_sqlite_bind_int(query, 2, tv.tv_usec) == false) {
res = false;
goto end;
}
if (lattutil_sqlite_bind_string(query, 3, msgtype) == false) {
res = false;
goto end;
}
if (lattutil_sqlite_bind_string(query, 4, msg) == false) {
res = false;
goto end;
}
res = lattutil_sqlite_exec(query);
end:
free(querystr);
lattutil_sqlite_query_free(&query);
return (res);
}
......@@ -29,22 +29,32 @@
#include "liblattutil.h"
static ssize_t _lattutil_log_stdio_debug(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_stdio_err(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_stdio_info(lattutil_log_t *, int,
const char *, ...);
static ssize_t _lattutil_log_stdio_warn(lattutil_log_t *, int,
const char *, ...);
static void _lattutil_log_stdio_close(lattutil_log_t *);
EXPORTED_SYM
bool
lattutil_log_stdio_init(lattutil_log_t *logp)
{
logp->ll_log_close = lattutil_log_stdio_close;
logp->ll_log_debug = lattutil_log_stdio_debug;
logp->ll_log_err = lattutil_log_stdio_err;
logp->ll_log_info = lattutil_log_stdio_info;
logp->ll_log_warn = lattutil_log_stdio_warn;
logp->ll_log_close = _lattutil_log_stdio_close;
logp->ll_log_debug = _lattutil_log_stdio_debug;
logp->ll_log_err = _lattutil_log_stdio_err;
logp->ll_log_info = _lattutil_log_stdio_info;
logp->ll_log_warn = _lattutil_log_stdio_warn;
return (true);
}
ssize_t
lattutil_log_stdio_debug(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_stdio_debug(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
va_list args;
......@@ -69,8 +79,8 @@ end:
return (len);
}
ssize_t
lattutil_log_stdio_err(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_stdio_err(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
va_list args;
......@@ -95,8 +105,8 @@ end:
return (len);
}
ssize_t
lattutil_log_stdio_info(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_stdio_info(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
va_list args;
......@@ -121,8 +131,8 @@ end:
return (len);
}
ssize_t
lattutil_log_stdio_warn(lattutil_log_t *logp, int verbose,
static ssize_t
_lattutil_log_stdio_warn(lattutil_log_t *logp, int verbose,
const char *fmt, ...)
{
va_list args;
......@@ -147,8 +157,8 @@ end:
return (len);
}
void
lattutil_log_stdio_close(lattutil_log_t *logp)
static void
_lattutil_log_stdio_close(lattutil_log_t *logp)
{
}
......@@ -31,17 +31,27 @@