Skip to content

Commit eaedf3b

Browse files
committed
Reindent safe.[ch]
1 parent e0b66d1 commit eaedf3b

File tree

2 files changed

+66
-61
lines changed

2 files changed

+66
-61
lines changed

src/safe.c

Lines changed: 61 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@
3131
/* Enable vasprintf */
3232
#define _GNU_SOURCE
3333

34-
3534
#include <stdarg.h>
3635
#include <stdio.h>
3736
#include <stdlib.h>
@@ -44,18 +43,19 @@
4443
#include <syslog.h>
4544

4645
/* From gateway.c */
47-
extern httpd * webserver;
46+
extern httpd *webserver;
4847

4948
void *
50-
safe_malloc (size_t size) {
51-
void * retval = NULL;
52-
retval = malloc(size);
53-
if (!retval) {
54-
debug(LOG_CRIT, "Failed to malloc %d bytes of memory: %s. Bailing out", size, strerror(errno));
55-
exit(1);
56-
}
49+
safe_malloc(size_t size)
50+
{
51+
void *retval = NULL;
52+
retval = malloc(size);
53+
if (!retval) {
54+
debug(LOG_CRIT, "Failed to malloc %d bytes of memory: %s. Bailing out", size, strerror(errno));
55+
exit(1);
56+
}
5757
memset(retval, 0, size);
58-
return (retval);
58+
return (retval);
5959
}
6060

6161
void *
@@ -70,59 +70,65 @@ safe_realloc(void *ptr, size_t newsize)
7070
return retval;
7171
}
7272

73-
char * safe_strdup(const char *s) {
74-
char * retval = NULL;
75-
if (!s) {
76-
debug(LOG_CRIT, "safe_strdup called with NULL which would have crashed strdup. Bailing out");
77-
exit(1);
78-
}
79-
retval = strdup(s);
80-
if (!retval) {
81-
debug(LOG_CRIT, "Failed to duplicate a string: %s. Bailing out", strerror(errno));
82-
exit(1);
83-
}
84-
return (retval);
73+
char *
74+
safe_strdup(const char *s)
75+
{
76+
char *retval = NULL;
77+
if (!s) {
78+
debug(LOG_CRIT, "safe_strdup called with NULL which would have crashed strdup. Bailing out");
79+
exit(1);
80+
}
81+
retval = strdup(s);
82+
if (!retval) {
83+
debug(LOG_CRIT, "Failed to duplicate a string: %s. Bailing out", strerror(errno));
84+
exit(1);
85+
}
86+
return (retval);
8587
}
8688

87-
int safe_asprintf(char **strp, const char *fmt, ...) {
88-
va_list ap;
89-
int retval;
89+
int
90+
safe_asprintf(char **strp, const char *fmt, ...)
91+
{
92+
va_list ap;
93+
int retval;
9094

91-
va_start(ap, fmt);
92-
retval = safe_vasprintf(strp, fmt, ap);
93-
va_end(ap);
95+
va_start(ap, fmt);
96+
retval = safe_vasprintf(strp, fmt, ap);
97+
va_end(ap);
9498

95-
return (retval);
99+
return (retval);
96100
}
97101

98-
int safe_vasprintf(char **strp, const char *fmt, va_list ap) {
99-
int retval;
102+
int
103+
safe_vasprintf(char **strp, const char *fmt, va_list ap)
104+
{
105+
int retval;
100106

101-
retval = vasprintf(strp, fmt, ap);
107+
retval = vasprintf(strp, fmt, ap);
102108

103-
if (retval == -1) {
104-
debug(LOG_CRIT, "Failed to vasprintf: %s. Bailing out", strerror(errno));
105-
exit (1);
106-
}
107-
return (retval);
109+
if (retval == -1) {
110+
debug(LOG_CRIT, "Failed to vasprintf: %s. Bailing out", strerror(errno));
111+
exit(1);
112+
}
113+
return (retval);
108114
}
109115

110-
pid_t safe_fork(void) {
111-
pid_t result;
112-
result = fork();
113-
114-
if (result == -1) {
115-
debug(LOG_CRIT, "Failed to fork: %s. Bailing out", strerror(errno));
116-
exit (1);
117-
}
118-
else if (result == 0) {
119-
/* I'm the child - do some cleanup */
120-
if (webserver) {
121-
close(webserver->serverSock);
122-
webserver = NULL;
123-
}
124-
}
125-
126-
return result;
127-
}
116+
pid_t
117+
safe_fork(void)
118+
{
119+
pid_t result;
120+
result = fork();
128121

122+
if (result == -1) {
123+
debug(LOG_CRIT, "Failed to fork: %s. Bailing out", strerror(errno));
124+
exit(1);
125+
} else if (result == 0) {
126+
/* I'm the child - do some cleanup */
127+
if (webserver) {
128+
close(webserver->serverSock);
129+
webserver = NULL;
130+
}
131+
}
132+
133+
return result;
134+
}

src/safe.h

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,9 @@
2828
#ifndef _SAFE_H_
2929
#define _SAFE_H_
3030

31-
#include <stdarg.h> /* For va_list */
32-
#include <sys/types.h> /* For fork */
33-
#include <unistd.h> /* For fork */
31+
#include <stdarg.h> /* For va_list */
32+
#include <sys/types.h> /* For fork */
33+
#include <unistd.h> /* For fork */
3434

3535
/** @brief Safe version of malloc
3636
*/
@@ -41,7 +41,7 @@ void *safe_realloc(void *, size_t);
4141

4242
/* @brief Safe version of strdup
4343
*/
44-
char * safe_strdup(const char *);
44+
char *safe_strdup(const char *);
4545

4646
/* @brief Safe version of asprintf
4747
*/
@@ -56,5 +56,4 @@ int safe_vasprintf(char **, const char *, va_list);
5656

5757
pid_t safe_fork(void);
5858

59-
#endif /* _SAFE_H_ */
60-
59+
#endif /* _SAFE_H_ */

0 commit comments

Comments
 (0)