aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPřemysl Eric Janouch <p@janouch.name>2020-10-22 02:09:22 +0200
committerPřemysl Eric Janouch <p@janouch.name>2020-10-22 02:19:59 +0200
commitddb3a60dcc83fc287eb00f02917a1240ce10c5e9 (patch)
treed1bd86e1267818547f7416bf1b69abbc14aae060
parent53bcebc2f0bae3ba0bbcefb849bdb0ede0ea4385 (diff)
downloadliberty-ddb3a60dcc83fc287eb00f02917a1240ce10c5e9.tar.gz
liberty-ddb3a60dcc83fc287eb00f02917a1240ce10c5e9.tar.xz
liberty-ddb3a60dcc83fc287eb00f02917a1240ce10c5e9.zip
Experimental support for building as C++origin/c++
So far with the following caveats: - Triggers -Wc99-designator - Compound literals are non-standard. - The setjmp/longjmp in the configuration parser might be an issue. - Perhaps others. It does not seem to be a good idea to use this library for C++ at all. Much of what it does is directly replaced by the STL.
-rw-r--r--liberty-proto.c33
-rw-r--r--liberty.c207
2 files changed, 147 insertions, 93 deletions
diff --git a/liberty-proto.c b/liberty-proto.c
index f7a62fd..5cf94ed 100644
--- a/liberty-proto.c
+++ b/liberty-proto.c
@@ -507,7 +507,7 @@ http_parse_upgrade (const char *upgrade, struct http_protocol **out)
case HTTP_T_WHITESPACE:
break;
case HTTP_T_TOKEN:
- proto = xcalloc (1, sizeof *proto);
+ proto = (struct http_protocol *) xcalloc (1, sizeof *proto);
proto->name = xstrdup (t.string.str);
LIST_APPEND_WITH_TAIL (list, tail, proto);
state = STATE_SLASH;
@@ -1002,7 +1002,7 @@ fcgi_nv_parser_push (struct fcgi_nv_parser *self, const void *data, size_t len)
if (self->input.len < self->name_len)
return;
- self->name = xmalloc (self->name_len + 1);
+ self->name = (char *) xmalloc (self->name_len + 1);
self->name[self->name_len] = '\0';
memcpy (self->name, self->input.str, self->name_len);
str_remove_slice (&self->input, 0, self->name_len);
@@ -1012,7 +1012,7 @@ fcgi_nv_parser_push (struct fcgi_nv_parser *self, const void *data, size_t len)
if (self->input.len < self->value_len)
return;
- self->value = xmalloc (self->value_len + 1);
+ self->value = (char *) xmalloc (self->value_len + 1);
self->value[self->value_len] = '\0';
memcpy (self->value, self->input.str, self->value_len);
str_remove_slice (&self->input, 0, self->value_len);
@@ -1050,7 +1050,7 @@ fcgi_nv_convert (struct str_map *map, struct str *output)
while (str_map_iter_next (&iter))
{
const char *name = iter.link->key;
- const char *value = iter.link->data;
+ const char *value = (const char *) iter.link->data;
size_t name_len = iter.link->key_length;
size_t value_len = strlen (value);
@@ -1227,7 +1227,7 @@ ws_parser_push (struct ws_parser *self, const void *data, size_t len)
self->reserved_1 = (u8 >> 6) & 1;
self->reserved_2 = (u8 >> 5) & 1;
self->reserved_3 = (u8 >> 4) & 1;
- self->opcode = u8 & 15;
+ self->opcode = (enum ws_opcode) (u8 & 15);
(void) msg_unpacker_u8 (&unpacker, &u8);
self->is_masked = (u8 >> 7) & 1;
@@ -1423,9 +1423,9 @@ mpd_client_make (struct poller *poller)
.socket = -1,
.read_buffer = str_make (),
.write_buffer = str_make (),
- .data = strv_make (),
.socket_event = poller_fd_make (poller, -1),
.timeout_timer = poller_timer_make (poller),
+ .data = strv_make (),
};
}
@@ -1464,7 +1464,7 @@ mpd_client_reset (struct mpd_client *self)
{
// Get rid of all pending tasks to release resources etc.
strv_reset (&self->data);
- struct mpd_response aborted = { .message_text = "Disconnected" };
+ struct mpd_response aborted = { .message_text = (char *) "Disconnected" };
while (self->tasks)
mpd_client_dispatch (self, &aborted);
@@ -1524,7 +1524,8 @@ mpd_client_parse_response (const char *p, struct mpd_response *response)
if (errno != 0 || end == p)
return false;
p = end;
- if (*p++ != ']' || *p++ != ' ' || *p++ != '{' || !(end = strchr (p, '}')))
+ if (*p++ != ']' || *p++ != ' ' || *p++ != '{'
+ || !(end = (char *) strchr (p, '}')))
return false;
response->current_command = xstrndup (p, end - p);
@@ -1624,7 +1625,7 @@ mpd_client_on_ready (const struct pollfd *pfd, void *user_data)
{
(void) pfd;
- struct mpd_client *self = user_data;
+ struct mpd_client *self = (struct mpd_client *) user_data;
if (socket_io_try_read (self->socket, &self->read_buffer) != SOCKET_IO_OK
|| !mpd_client_process_input (self)
|| socket_io_try_write (self->socket, &self->write_buffer) != SOCKET_IO_OK)
@@ -1676,7 +1677,8 @@ mpd_client_add_task
// later flushed if an early ACK or OK arrives).
hard_assert (!self->in_list);
- struct mpd_client_task *task = xcalloc (1, sizeof *self);
+ struct mpd_client_task *task =
+ (struct mpd_client_task *) xcalloc (1, sizeof *self);
task->callback = cb;
task->user_data = user_data;
LIST_APPEND_WITH_TAIL (self->tasks, self->tasks_tail, task);
@@ -1794,7 +1796,7 @@ mpd_client_on_idle_return (const struct mpd_response *response,
{
(void) response;
- struct mpd_client *self = user_data;
+ struct mpd_client *self = (struct mpd_client *) user_data;
unsigned subsystems = 0;
for (size_t i = 0; i < data->len; i++)
{
@@ -1817,7 +1819,7 @@ static void mpd_client_idle (struct mpd_client *self, unsigned subsystems);
static void
mpd_client_on_timeout (void *user_data)
{
- struct mpd_client *self = user_data;
+ struct mpd_client *self = (struct mpd_client *) user_data;
// Abort and immediately restore the current idle so that MPD doesn't
// disconnect us, even though the documentation says this won't happen.
@@ -1885,7 +1887,7 @@ mpd_client_destroy_connector (struct mpd_client *self)
static void
mpd_client_on_connector_failure (void *user_data)
{
- struct mpd_client *self = user_data;
+ struct mpd_client *self = (struct mpd_client *) user_data;
mpd_client_destroy_connector (self);
mpd_client_fail (self);
}
@@ -1896,7 +1898,7 @@ mpd_client_on_connector_connected
{
(void) host;
- struct mpd_client *self = user_data;
+ struct mpd_client *self = (struct mpd_client *) user_data;
mpd_client_destroy_connector (self);
mpd_client_finish_connection (self, socket);
}
@@ -1940,7 +1942,8 @@ mpd_client_connect (struct mpd_client *self, const char *address,
if (strchr (address, '/'))
return mpd_client_connect_unix (self, address, e);
- struct connector *connector = xmalloc (sizeof *connector);
+ struct connector *connector =
+ (struct connector *) xmalloc (sizeof *connector);
connector_init (connector, self->poller);
self->connector = connector;
diff --git a/liberty.c b/liberty.c
index d3c6c25..a51d932 100644
--- a/liberty.c
+++ b/liberty.c
@@ -117,7 +117,8 @@ extern char **environ;
#define CONTAINER_OF(pointer, type, member) \
((type *) ((char *) pointer - offsetof (type, member)))
-char *liberty = "They who can give up essential liberty to obtain a little "
+const char *liberty =
+ "They who can give up essential liberty to obtain a little "
"temporary safety deserve neither liberty nor safety.";
// --- Logging -----------------------------------------------------------------
@@ -287,7 +288,7 @@ xreallocarray (void *o, size_t n, size_t m)
static char *
xstrdup (const char *s)
{
- return strcpy (xmalloc (strlen (s) + 1), s);
+ return strcpy ((char *) xmalloc (strlen (s) + 1), s);
}
static char *
@@ -297,7 +298,7 @@ xstrndup (const char *s, size_t n)
if (n > size)
n = size;
- char *copy = xmalloc (n + 1);
+ char *copy = (char *) xmalloc (n + 1);
memcpy (copy, s, n);
copy[n] = '\0';
return copy;
@@ -310,14 +311,15 @@ xstrndup (const char *s, size_t n)
#define ARRAY(type, name) type *name; size_t name ## _len, name ## _alloc;
#define ARRAY_INIT_SIZED(a, n) \
BLOCK_START \
- (a) = xcalloc (sizeof *(a), (a ## _alloc) = (n)); \
+ (a) = (type *) xcalloc (sizeof *(a), (a ## _alloc) = (n)); \
(a ## _len) = 0; \
BLOCK_END
#define ARRAY_INIT(a) ARRAY_INIT_SIZED (a, 16)
#define ARRAY_RESERVE(a, n) \
BLOCK_START \
while ((a ## _alloc) - (a ## _len) < n) \
- (a) = xreallocarray ((a), sizeof *(a), (a ## _alloc) <<= 1); \
+ (a) = (type *) xreallocarray ((a), \
+ sizeof *(a), (a ## _alloc) <<= 1); \
BLOCK_END
// --- Double-linked list helpers ----------------------------------------------
@@ -392,7 +394,7 @@ strv_make (void)
struct strv self;
self.alloc = 4;
self.len = 0;
- self.vector = xcalloc (sizeof *self.vector, self.alloc);
+ self.vector = (char **) xcalloc (sizeof *self.vector, self.alloc);
return self;
}
@@ -419,7 +421,7 @@ strv_append_owned (struct strv *self, char *s)
{
self->vector[self->len] = s;
if (++self->len >= self->alloc)
- self->vector = xreallocarray (self->vector,
+ self->vector = (char **) xreallocarray (self->vector,
sizeof *self->vector, (self->alloc <<= 1));
self->vector[self->len] = NULL;
}
@@ -492,7 +494,7 @@ str_make (void)
struct str self;
self.alloc = 16;
self.len = 0;
- self.str = strcpy (xmalloc (self.alloc), "");
+ self.str = strcpy ((char *) xmalloc (self.alloc), "");
return self;
}
@@ -529,7 +531,7 @@ str_reserve (struct str *self, size_t n)
while (new_alloc <= self->len + n)
new_alloc <<= 1;
if (new_alloc != self->alloc)
- self->str = xrealloc (self->str, (self->alloc = new_alloc));
+ self->str = (char *) xrealloc (self->str, (self->alloc = new_alloc));
}
static void
@@ -608,7 +610,7 @@ str_remove_slice (struct str *self, size_t start, size_t length)
// Shrink the string if the allocation becomes way too large
if (self->alloc >= STR_SHRINK_THRESHOLD && self->len < (self->alloc >> 2))
- self->str = xrealloc (self->str, self->alloc >>= 2);
+ self->str = (char *) xrealloc (self->str, self->alloc >>= 2);
}
// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
@@ -622,7 +624,11 @@ str_pack_u8 (struct str *self, uint8_t x)
static void
str_pack_u16 (struct str *self, uint16_t x)
{
- uint8_t tmp[2] = { x >> 8, x };
+ uint8_t tmp[2] =
+ {
+ (uint8_t) (x >> 8),
+ (uint8_t) x
+ };
str_append_data (self, tmp, sizeof tmp);
}
@@ -630,7 +636,13 @@ static void
str_pack_u32 (struct str *self, uint32_t x)
{
uint32_t u = x;
- uint8_t tmp[4] = { u >> 24, u >> 16, u >> 8, u };
+ uint8_t tmp[4] =
+ {
+ (uint8_t) (u >> 24),
+ (uint8_t) (u >> 16),
+ (uint8_t) (u >> 8),
+ (uint8_t) u
+ };
str_append_data (self, tmp, sizeof tmp);
}
@@ -638,7 +650,16 @@ static void
str_pack_u64 (struct str *self, uint64_t x)
{
uint8_t tmp[8] =
- { x >> 56, x >> 48, x >> 40, x >> 32, x >> 24, x >> 16, x >> 8, x };
+ {
+ (uint8_t) (x >> 56),
+ (uint8_t) (x >> 48),
+ (uint8_t) (x >> 40),
+ (uint8_t) (x >> 32),
+ (uint8_t) (x >> 24),
+ (uint8_t) (x >> 16),
+ (uint8_t) (x >> 8),
+ (uint8_t) x
+ };
str_append_data (self, tmp, sizeof tmp);
}
@@ -672,8 +693,8 @@ error_set (struct error **e, const char *message, ...)
hard_assert (size >= 0);
- struct error *tmp = xmalloc (sizeof *tmp);
- tmp->message = xmalloc (size + 1);
+ struct error *tmp = (struct error *) xmalloc (sizeof *tmp);
+ tmp->message = (char *) xmalloc (size + 1);
va_start (ap, message);
size = vsnprintf (tmp->message, size + 1, message, ap);
@@ -778,7 +799,7 @@ random_bytes (void *output, size_t len, struct error **e)
// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
-static unsigned char g_siphash_key[16] = "SipHash 2-4 key!";
+static unsigned char g_siphash_key[16] = "SipHash 2-4 key" /* \0 */;
static inline void
siphash_wrapper_randomize (void)
@@ -790,7 +811,7 @@ siphash_wrapper_randomize (void)
static inline uint64_t
siphash_wrapper (const void *m, size_t len)
{
- return siphash (g_siphash_key, m, len);
+ return siphash (g_siphash_key, (const unsigned char *) m, len);
}
// --- String hash map ---------------------------------------------------------
@@ -832,7 +853,7 @@ str_map_make (str_map_free_fn free)
self.len = 0;
self.free = free;
self.key_xfrm = NULL;
- self.map = xcalloc (self.alloc, sizeof *self.map);
+ self.map = (struct str_map_link **) xcalloc (self.alloc, sizeof *self.map);
self.shrink_lock = false;
return self;
}
@@ -888,7 +909,8 @@ str_map_resize (struct str_map *self, size_t new_size)
size_t mask = new_size - 1;
self->alloc = new_size;
- self->map = xcalloc (self->alloc, sizeof *self->map);
+ self->map =
+ (struct str_map_link **) xcalloc (self->alloc, sizeof *self->map);
for (i = 0; i < old_size; i++)
{
struct str_map_link *iter = old_map[i], *next_iter;
@@ -958,7 +980,8 @@ str_map_set_real (struct str_map *self, const char *key, void *value)
// Link in a new element for the given <key, value> pair
size_t key_length = strlen (key);
- struct str_map_link *link = xmalloc (sizeof *link + key_length + 1);
+ struct str_map_link *link =
+ (struct str_map_link *) xmalloc (sizeof *link + key_length + 1);
link->data = value;
link->key_length = key_length;
memcpy (link->key, key, key_length + 1);
@@ -1161,7 +1184,7 @@ async_cancel (struct async *self)
static void
async_cleanup (void *user_data)
{
- struct async *self = user_data;
+ struct async *self = (struct async *) user_data;
hard_assert (!pthread_mutex_lock (&self->manager->lock));
LIST_UNLINK (self->manager->running, self);
@@ -1177,7 +1200,7 @@ async_routine (void *user_data)
{
// Beware that we mustn't trigger any cancellation point before we set up
// the cleanup handler, otherwise we'd need to disable it first
- struct async *self = user_data;
+ struct async *self = (struct async *) user_data;
pthread_cleanup_push (async_cleanup, self);
self->execute (self);
@@ -1386,7 +1409,8 @@ poller_timers_make (void)
struct poller_timers self;
self.alloc = POLLER_MIN_ALLOC;
self.len = 0;
- self.heap = xmalloc (self.alloc * sizeof *self.heap);
+ self.heap =
+ (struct poller_timer **) xmalloc (self.alloc * sizeof *self.heap);
return self;
}
@@ -1496,7 +1520,7 @@ poller_timers_set (struct poller_timers *self, struct poller_timer *timer)
}
if (self->len == self->alloc)
- self->heap = xreallocarray (self->heap,
+ self->heap = (struct poller_timer **) xreallocarray (self->heap,
self->alloc <<= 1, sizeof *self->heap);
self->heap[self->len] = timer;
timer->index = self->len;
@@ -1573,9 +1597,10 @@ poller_init (struct poller *self)
self->len = 0;
self->alloc = POLLER_MIN_ALLOC;
- self->fds = xcalloc (self->alloc, sizeof *self->fds);
- self->dummy = xcalloc (self->alloc, sizeof *self->dummy);
- self->revents = xcalloc (self->alloc, sizeof *self->revents);
+ self->fds = (struct poller_fd **) xcalloc (self->alloc, sizeof *self->fds);
+ self->dummy = (int *) xcalloc (self->alloc, sizeof *self->dummy);
+ self->revents = (struct epoll_event *)
+ xcalloc (self->alloc, sizeof *self->revents);
self->revents_len = 0;
poller_common_init (&self->common, self);
@@ -1588,7 +1613,7 @@ poller_free (struct poller *self)
{
struct poller_fd *fd = self->fds[i];
hard_assert (epoll_ctl (self->epoll_fd,
- EPOLL_CTL_DEL, fd->fd, (void *) "") != -1);
+ EPOLL_CTL_DEL, fd->fd, (struct epoll_event *) "") != -1);
}
poller_common_free (&self->common);
@@ -1608,11 +1633,11 @@ poller_ensure_space (struct poller *self)
self->alloc <<= 1;
hard_assert (self->alloc != 0);
- self->revents = xreallocarray
+ self->revents = (struct epoll_event *) xreallocarray
(self->revents, sizeof *self->revents, self->alloc);
- self->fds = xreallocarray
+ self->fds = (struct poller_fd **) xreallocarray
(self->fds, sizeof *self->fds, self->alloc);
- self->dummy = xreallocarray
+ self->dummy = (int *) xreallocarray
(self->dummy, sizeof *self->dummy, self->alloc);
}
@@ -1664,8 +1689,12 @@ poller_set (struct poller *self, struct poller_fd *fd)
static int
poller_compare_fds (const void *ax, const void *bx)
{
- const struct epoll_event *ay = ax, *by = bx;
- struct poller_fd *a = ay->data.ptr, *b = by->data.ptr;
+ const struct epoll_event
+ *ay = (const struct epoll_event *) ax,
+ *by = (const struct epoll_event *) bx;
+ struct poller_fd
+ *a = (struct poller_fd *) ay->data.ptr,
+ *b = (struct poller_fd *) by->data.ptr;
return a->fd - b->fd;
}
@@ -1676,7 +1705,7 @@ poller_remove_from_dispatch (struct poller *self, const struct poller_fd *fd)
return;
struct epoll_event key = { .data.ptr = (void *) fd }, *fd_event;
- if ((fd_event = bsearch (&key, self->revents,
+ if ((fd_event = (struct epoll_event *) bsearch (&key, self->revents,
self->revents_len, sizeof *self->revents, poller_compare_fds)))
{
fd_event->events = -1;
@@ -1699,7 +1728,7 @@ poller_remove_at_index (struct poller *self, size_t index)
poller_remove_from_dispatch (self, fd);
if (!fd->closed)
hard_assert (epoll_ctl (self->epoll_fd,
- EPOLL_CTL_DEL, fd->fd, (void *) "") != -1);
+ EPOLL_CTL_DEL, fd->fd, (struct epoll_event *) "") != -1);
if (index != --self->len)
{
@@ -1735,7 +1764,7 @@ poller_run (struct poller *self)
if (revents->events == (uint32_t) -1)
continue;
- struct poller_fd *fd = revents->data.ptr;
+ struct poller_fd *fd = (struct poller_fd *) revents->data.ptr;
hard_assert (fd->index != -1);
struct pollfd pfd;
@@ -1778,8 +1807,9 @@ poller_init (struct poller *self)
self->len = 0;
self->alloc = POLLER_MIN_ALLOC;
- self->fds = xcalloc (self->alloc, sizeof *self->fds);
- self->revents = xcalloc (self->alloc, sizeof *self->revents);
+ self->fds = (struct poller_fd **) xcalloc (self->alloc, sizeof *self->fds);
+ self->revents = (struct kevent *)
+ xcalloc (self->alloc, sizeof *self->revents);
self->revents_len = 0;
poller_common_init (&self->common, self);
}
@@ -1802,9 +1832,9 @@ poller_ensure_space (struct poller *self)
self->alloc <<= 1;
hard_assert (self->alloc != 0);
- self->revents = xreallocarray
+ self->revents = (struct kevent *) xreallocarray
(self->revents, sizeof *self->revents, self->alloc);
- self->fds = xreallocarray
+ self->fds = (struct poller_fd **) xreallocarray
(self->fds, sizeof *self->fds, self->alloc);
}
@@ -2004,8 +2034,9 @@ poller_init (struct poller *self)
{
self->alloc = POLLER_MIN_ALLOC;
self->len = 0;
- self->fds = xcalloc (self->alloc, sizeof *self->fds);
- self->fds_data = xcalloc (self->alloc, sizeof *self->fds_data);
+ self->fds = (struct pollfd **) xcalloc (self->alloc, sizeof *self->fds);
+ self->fds_data = (struct poller_fd **)
+ xcalloc (self->alloc, sizeof *self->fds_data);
poller_common_init (&self->common, self);
self->dispatch_next = -1;
}
@@ -2025,8 +2056,9 @@ poller_ensure_space (struct poller *self)
return;
self->alloc <<= 1;
- self->fds = xreallocarray (self->fds, sizeof *self->fds, self->alloc);
- self->fds_data = xreallocarray
+ self->fds = (struct pollfd *)
+ xreallocarray (self->fds, sizeof *self->fds, self->alloc);
+ self->fds_data = (struct poller_fd **) xreallocarray
(self->fds_data, sizeof *self->fds_data, self->alloc);
}
@@ -2310,7 +2342,8 @@ static struct async_getaddrinfo *
async_getaddrinfo (struct async_manager *manager,
const char *host, const char *service, const struct addrinfo *hints)
{
- struct async_getaddrinfo *self = xcalloc (1, sizeof *self);
+ struct async_getaddrinfo *self =
+ (struct async_getaddrinfo *) xcalloc (1, sizeof *self);
self->async = async_make (manager);
if (host) self->host = xstrdup (host);
@@ -2374,10 +2407,11 @@ static struct async_getnameinfo *
async_getnameinfo (struct async_manager *manager,
const struct sockaddr *sa, socklen_t sa_len, int flags)
{
- struct async_getnameinfo *self = xcalloc (1, sizeof *self);
+ struct async_getnameinfo *self =
+ (struct async_getnameinfo *) xcalloc (1, sizeof *self);
self->async = async_make (manager);
- self->address = memcpy (xmalloc (sa_len), sa, sa_len);
+ self->address = (struct sockaddr *) memcpy (xmalloc (sa_len), sa, sa_len);
self->address_len = sa_len;
self->flags = flags;
@@ -2542,7 +2576,12 @@ struct msg_unpacker
static struct msg_unpacker
msg_unpacker_make (const void *data, size_t len)
{
- return (struct msg_unpacker) { .data = data, .len = len, .offset = 0 };
+ return (struct msg_unpacker)
+ {
+ .data = (const char *) data,
+ .offset = 0,
+ .len = len
+ };
}
static size_t
@@ -2631,8 +2670,16 @@ msg_writer_flush (struct msg_writer *self, size_t *len)
{
// Update the message length
uint64_t x = self->buf.len;
- uint8_t tmp[8] =
- { x >> 56, x >> 48, x >> 40, x >> 32, x >> 24, x >> 16, x >> 8, x };
+ uint8_t tmp[8] = {
+ (uint8_t) (x >> 56),
+ (uint8_t) (x >> 48),
+ (uint8_t) (x >> 40),
+ (uint8_t) (x >> 32),
+ (uint8_t) (x >> 24),
+ (uint8_t) (x >> 16),
+ (uint8_t) (x >> 8),
+ (uint8_t) x
+ };
memcpy (self->buf.str, tmp, sizeof tmp);
*len = x;
@@ -2911,7 +2958,7 @@ base64_encode (const void *data, size_t len, struct str *output)
const char *alphabet =
"ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/";
- const uint8_t *p = data;
+ const uint8_t *p = (const uint8_t *) data;
size_t n_groups = len / 3;
size_t tail = len - n_groups * 3;
uint32_t group;
@@ -2948,10 +2995,10 @@ base64_encode (const void *data, size_t len, struct str *output)
// --- Utilities ---------------------------------------------------------------
static void
-cstr_set (char **s, char *new)
+cstr_set (char **s, char *new_)
{
free (*s);
- *s = new;
+ *s = new_;
}
static void
@@ -3029,7 +3076,7 @@ iconv_xstrdup (iconv_t conv, char *in, size_t in_len, size_t *out_len)
char *buf, *buf_ptr;
size_t out_left, buf_alloc;
- buf = buf_ptr = xmalloc (out_left = buf_alloc = 64);
+ buf = buf_ptr = (char *) xmalloc (out_left = buf_alloc = 64);
char *in_ptr = in;
if (in_len == (size_t) -1)
@@ -3045,7 +3092,7 @@ iconv_xstrdup (iconv_t conv, char *in, size_t in_len, size_t *out_len)
return NULL;
}
out_left += buf_alloc;
- char *new_buf = xrealloc (buf, buf_alloc <<= 1);
+ char *new_buf = (char *) xrealloc (buf, buf_alloc <<= 1);
buf_ptr += new_buf - buf;
buf = new_buf;
}
@@ -3126,8 +3173,8 @@ lock_pid_file (const char *path, struct error **e)
struct flock lock =
{
.l_type = F_WRLCK,
- .l_start = 0,
.l_whence = SEEK_SET,
+ .l_start = 0,
.l_len = 0,
};
if (fcntl (fd, F_SETLK, &lock))
@@ -3326,7 +3373,7 @@ resolve_relative_runtime_filename (const char *filename)
/// path can reside in a system-wide directory with no risk of a conflict.
/// However, we have to take care about permissions. Do we even need this?
static char *
-resolve_relative_runtime_template (const char *template)
+resolve_relative_runtime_template (const char *template_)
{
struct str path = str_make ();
const char *runtime_dir = getenv ("XDG_RUNTIME_DIR");
@@ -3338,7 +3385,7 @@ resolve_relative_runtime_template (const char *template)
else
str_append_printf (&path, "/tmp/%s.%d", PROGRAM_NAME, geteuid ());
- str_append_printf (&path, "/%s", template);
+ str_append_printf (&path, "/%s", template_);
return resolve_relative_runtime_filename_finish (path);
}
@@ -3360,9 +3407,9 @@ try_expand_tilde (const char *filename)
struct passwd pwd, *success = NULL;
char *user = xstrndup (filename, until_slash);
- char *buf = xmalloc (buf_len);
+ char *buf = (char *) xmalloc (buf_len);
while (getpwnam_r (user, &pwd, buf, buf_len, &success) == ERANGE)
- buf = xrealloc (buf, buf_len <<= 1);
+ buf = (char *) xrealloc (buf, buf_len <<= 1);
free (user);
char *result = NULL;
@@ -3438,7 +3485,7 @@ xssl_get_error (SSL *ssl, int result, const char **error_info)
static regex_t *
regex_compile (const char *regex, int flags, struct error **e)
{
- regex_t *re = xmalloc (sizeof *re);
+ regex_t *re = (regex_t *) xmalloc (sizeof *re);
int err = regcomp (re, regex, flags);
if (!err)
return re;
@@ -3454,7 +3501,7 @@ regex_compile (const char *regex, int flags, struct error **e)
static void
regex_free (void *regex)
{
- regfree (regex);
+ regfree ((regex_t *) regex);
free (regex);
}
@@ -3473,7 +3520,7 @@ static bool
regex_cache_match (struct str_map *cache, const char *regex, int flags,
const char *s, struct error **e)
{
- regex_t *re = str_map_find (cache, regex);
+ regex_t *re = (regex_t *) str_map_find (cache, regex);
if (!re)
{
re = regex_compile (regex, flags, e);
@@ -3743,7 +3790,7 @@ opt_handler_make (int argc, char **argv,
self.opts = opts;
self.opts_len = len;
- self.options = xcalloc (len + 1, sizeof *self.options);
+ self.options = (struct option *) xcalloc (len + 1, sizeof *self.options);
struct str opt_string = str_make ();
for (size_t i = 0; i < len; i++)
@@ -3921,7 +3968,7 @@ test_add_internal (struct test *self, const char *name, size_t fixture_size,
hard_assert (test != NULL);
hard_assert (name != NULL);
- struct test_unit *unit = xcalloc (1, sizeof *unit);
+ struct test_unit *unit = (struct test_unit *) xcalloc (1, sizeof *unit);
unit->name = xstrdup (name);
unit->fixture_size = fixture_size;
unit->user_data = user_data;
@@ -4074,7 +4121,8 @@ struct connector_target
static struct connector_target *
connector_target_new (void)
{
- struct connector_target *self = xcalloc (1, sizeof *self);
+ struct connector_target *self =
+ (struct connector_target *) xcalloc (1, sizeof *self);
return self;
}
@@ -4287,7 +4335,7 @@ connector_free (struct connector *self)
static void
connector_on_getaddrinfo (int err, struct addrinfo *results, void *user_data)
{
- struct connector_target *self = user_data;
+ struct connector_target *self = (struct connector_target *) user_data;
if (err)
{
@@ -4540,7 +4588,8 @@ config_item_move (struct config_item *self, struct config_item *source)
static struct config_item *
config_item_new (enum config_item_type type)
{
- struct config_item *self = xcalloc (1, sizeof *self);
+ struct config_item *self =
+ (struct config_item *) xcalloc (1, sizeof *self);
self->type = type;
return self;
}
@@ -4680,7 +4729,8 @@ config_item_get (struct config_item *self, const char *path, struct error **e)
const char *key = v.vector[i];
if (!*key)
error_set (e, "empty path element");
- else if (!(self = str_map_find (&self->value.object, key)))
+ else if (!(self = (struct config_item *)
+ str_map_find (&self->value.object, key)))
error_set (e, "`%s' not found in object", key);
else if (++i == v.len)
result = self;
@@ -4811,7 +4861,7 @@ config_item_write_object_innards
struct str_map_iter iter = str_map_iter_make (&object->value.object);
struct config_item *value;
- while ((value = str_map_iter_next (&iter)))
+ while ((value = (struct config_item *) str_map_iter_next (&iter)))
config_item_write_kv_pair (self, iter.link->key, value);
}
@@ -5377,11 +5427,11 @@ config_read_from_file (const char *filename, struct error **e)
{
struct config_item *root = NULL;
+ struct error *error = NULL;
struct str data = str_make ();
if (!read_file (filename, &data, e))
goto end;
- struct error *error = NULL;
if (!(root = config_item_parse (data.str, data.len, false, &error)))
{
error_set (e, "parse error in `%s': %s", filename, error->message);
@@ -5401,12 +5451,12 @@ config_schema_initialize_item (struct config_schema *schema,
struct error **e)
{
hard_assert (parent->type == CONFIG_ITEM_OBJECT);
- struct config_item *item =
+ struct config_item *item = (struct config_item *)
str_map_find (&parent->value.object, schema->name);
+ struct error *error = NULL;
if (item)
{
- struct error *error = NULL;
item->user_data = user_data;
if (config_item_validate_by_schema (item, schema, &error))
goto keep_current;
@@ -5414,9 +5464,9 @@ config_schema_initialize_item (struct config_schema *schema,
error_set (warning, "resetting configuration item "
"`%s' to default: %s", schema->name, error->message);
error_free (error);
+ error = NULL;
}
- struct error *error = NULL;
if (schema->default_)
item = config_item_parse
(schema->default_, strlen (schema->default_), true, &error);
@@ -5479,7 +5529,7 @@ config_schema_call_changed (struct config_item *item)
{
struct str_map_iter iter = str_map_iter_make (&item->value.object);
struct config_item *child;
- while ((child = str_map_iter_next (&iter)))
+ while ((child = (struct config_item *) str_map_iter_next (&iter)))
config_schema_call_changed (child);
}
else if (item->schema && item->schema->on_change)
@@ -5532,7 +5582,8 @@ static void
config_register_module (struct config *self,
const char *name, config_module_load_fn loader, void *user_data)
{
- struct config_module *module = xcalloc (1, sizeof *module);
+ struct config_module *module =
+ (struct config_module *) xcalloc (1, sizeof *module);
module->name = xstrdup (name);
module->loader = loader;
module->user_data = user_data;
@@ -5550,9 +5601,9 @@ config_load (struct config *self, struct config_item *root)
struct str_map_iter iter = str_map_iter_make (&self->modules);
struct config_module *module;
- while ((module = str_map_iter_next (&iter)))
+ while ((module = (struct config_module *) str_map_iter_next (&iter)))
{
- struct config_item *subtree = str_map_find
+ struct config_item *subtree = (struct config_item *) str_map_find
(&root->value.object, module->name);
// Silently fix inputs that only a lunatic user could create
if (!subtree || subtree->type != CONFIG_ITEM_OBJECT)