bzr branch
http://bzr.recompile.se/loggerhead/mandos/trunk
| 21
by Teddy Hogeborn * Makefile (CFLAGS): Changed to use $(WARN), $(DEBUG), $(COVERAGE) and | 1 | /*  -*- coding: utf-8 -*- */
 | 
| 2 | /*
 | |
| 3 |  * Mandos plugin runner - Run Mandos plugins
 | |
| 4 |  *
 | |
| 28
by Teddy Hogeborn * server.conf: New file. | 5 |  * Copyright © 2007-2008 Teddy Hogeborn & Björn Påhlsson
 | 
| 21
by Teddy Hogeborn * Makefile (CFLAGS): Changed to use $(WARN), $(DEBUG), $(COVERAGE) and | 6 |  * 
 | 
| 7 |  * This program is free software: you can redistribute it and/or
 | |
| 8 |  * modify it under the terms of the GNU General Public License as
 | |
| 9 |  * published by the Free Software Foundation, either version 3 of the
 | |
| 10 |  * License, or (at your option) any later version.
 | |
| 11 |  * 
 | |
| 12 |  * This program is distributed in the hope that it will be useful, but
 | |
| 13 |  * WITHOUT ANY WARRANTY; without even the implied warranty of
 | |
| 14 |  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 | |
| 15 |  * General Public License for more details.
 | |
| 16 |  * 
 | |
| 17 |  * You should have received a copy of the GNU General Public License
 | |
| 18 |  * along with this program.  If not, see
 | |
| 19 |  * <http://www.gnu.org/licenses/>.
 | |
| 20 |  * 
 | |
| 28
by Teddy Hogeborn * server.conf: New file. | 21 |  * Contact the authors at <mandos@fukt.bsnet.se>.
 | 
| 21
by Teddy Hogeborn * Makefile (CFLAGS): Changed to use $(WARN), $(DEBUG), $(COVERAGE) and | 22 |  */
 | 
| 23 | ||
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 24 | #define _GNU_SOURCE /* TEMP_FAILURE_RETRY(), getline(), | 
| 25 | 				   asprintf() */
 | |
| 24.1.26
by Björn Påhlsson tally count of used symbols | 26 | #include <stddef.h> /* size_t, NULL */ | 
| 27 | #include <stdlib.h> /* malloc(), exit(), EXIT_FAILURE, | |
| 28 | EXIT_SUCCESS, realloc() */ | |
| 29 | #include <stdbool.h> /* bool, true, false */ | |
| 30 | #include <stdio.h> /* perror, popen(), fileno(), | |
| 31 | fprintf(), stderr, STDOUT_FILENO */ | |
| 32 | #include <sys/types.h> /* DIR, opendir(), stat(), struct | |
| 33 | stat, waitpid(), WIFEXITED(), | |
| 34 | WEXITSTATUS(), wait(), pid_t, | |
| 35 | uid_t, gid_t, getuid(), getgid(), | |
| 36 | dirfd() */ | |
| 37 | #include <sys/select.h> /* fd_set, select(), FD_ZERO(), | |
| 38 | FD_SET(), FD_ISSET(), FD_CLR */ | |
| 39 | #include <sys/wait.h> /* wait(), waitpid(), WIFEXITED(), | |
| 40 | WEXITSTATUS() */ | |
| 41 | #include <sys/stat.h> /* struct stat, stat(), S_ISREG() */ | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 42 | #include <iso646.h> /* and, or, not */ | 
| 43 | #include <dirent.h> /* DIR, struct dirent, opendir(), | |
| 24.1.26
by Björn Påhlsson tally count of used symbols | 44 | readdir(), closedir(), dirfd() */ | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 45 | #include <unistd.h> /* struct stat, stat(), S_ISREG(), | 
| 24.1.26
by Björn Påhlsson tally count of used symbols | 46 | fcntl(), setuid(), setgid(), | 
| 47 | F_GETFD, F_SETFD, FD_CLOEXEC, | |
| 48 | access(), pipe(), fork(), close() | |
| 49 | dup2, STDOUT_FILENO, _exit(), | |
| 50 | execv(), write(), read(), | |
| 51 | close() */ | |
| 52 | #include <fcntl.h> /* fcntl(), F_GETFD, F_SETFD, | |
| 53 | FD_CLOEXEC */ | |
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 54 | #include <string.h> /* strsep, strlen(), asprintf() */ | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 55 | #include <errno.h> /* errno */ | 
| 24.1.26
by Björn Påhlsson tally count of used symbols | 56 | #include <argp.h> /* struct argp_option, struct | 
| 57 | argp_state, struct argp, | |
| 58 | argp_parse(), ARGP_ERR_UNKNOWN, | |
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 59 | ARGP_KEY_END, ARGP_KEY_ARG, | 
| 60 | error_t */ | |
| 24.1.26
by Björn Påhlsson tally count of used symbols | 61 | #include <signal.h> /* struct sigaction, sigemptyset(), | 
| 62 | sigaddset(), sigaction(), | |
| 63 | sigprocmask(), SIG_BLOCK, SIGCHLD, | |
| 64 | SIG_UNBLOCK, kill() */ | |
| 65 | #include <errno.h> /* errno, EBADF */ | |
| 13
by Björn Påhlsson Added following support: | 66 | |
| 37
by Teddy Hogeborn Non-tested commit for merge purposes. | 67 | #define BUFFER_SIZE 256
 | 
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 68 | |
| 69 | #define PDIR "/lib/mandos/plugins.d"
 | |
| 70 | #define AFILE "/conf/conf.d/mandos/plugin-runner.conf"
 | |
| 37
by Teddy Hogeborn Non-tested commit for merge purposes. | 71 | |
| 74
by Teddy Hogeborn * Makefile (PREFIX, CONFDIR): New. | 72 | const char *argp_program_version = "plugin-runner 1.0"; | 
| 24.1.35
by Björn Påhlsson version 1.0 | 73 | const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>"; | 
| 74 | ||
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 75 | struct plugin; | 
| 76 | ||
| 77 | typedef struct plugin{ | |
| 78 | char *name; /* can be NULL or any plugin name */ | |
| 79 | char **argv; | |
| 80 | int argc; | |
| 81 | char **environ; | |
| 82 | int envc; | |
| 83 | bool disabled; | |
| 84 | ||
| 85 | /* Variables used for running processes*/ | |
| 13
by Björn Påhlsson Added following support: | 86 | pid_t pid; | 
| 87 | int fd; | |
| 88 | char *buffer; | |
| 21
by Teddy Hogeborn * Makefile (CFLAGS): Changed to use $(WARN), $(DEBUG), $(COVERAGE) and | 89 | size_t buffer_size; | 
| 90 | size_t buffer_length; | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 91 | bool eof; | 
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 92 | volatile bool completed; | 
| 93 | volatile int status; | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 94 | struct plugin *next; | 
| 95 | } plugin; | |
| 96 | ||
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 97 | static plugin *plugin_list = NULL; | 
| 98 | ||
| 99 | /* Gets a existing plugin based on name,
 | |
| 100 |    or if none is found, creates a new one */
 | |
| 101 | static plugin *getplugin(char *name){ | |
| 102 | /* Check for exiting plugin with that name */ | |
| 103 | for (plugin *p = plugin_list; p != NULL; p = p->next){ | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 104 | if ((p->name == name) | 
| 105 | or (p->name and name and (strcmp(p->name, name) == 0))){ | |
| 106 | return p; | |
| 107 | } | |
| 108 | } | |
| 109 | /* Create a new plugin */ | |
| 110 | plugin *new_plugin = malloc(sizeof(plugin)); | |
| 111 | if (new_plugin == NULL){ | |
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 112 | return NULL; | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 113 | } | 
| 24.1.62
by Björn Påhlsson merge + small bugfix | 114 | char *copy_name = NULL; | 
| 115 | if(name != NULL){ | |
| 116 | copy_name = strdup(name); | |
| 98
by Teddy Hogeborn * plugin-runner.c (getplugin): Only copy "name" if not NULL. Free | 117 | if(copy_name == NULL){ | 
| 118 | return NULL; | |
| 119 | } | |
| 24.1.54
by Björn Påhlsson plugin-runner | 120 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 121 | |
| 24.1.54
by Björn Påhlsson plugin-runner | 122 | *new_plugin = (plugin) { .name = copy_name, | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 123 | .argc = 1, | 
| 124 | .disabled = false, | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 125 | .next = plugin_list }; | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 126 |   
 | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 127 | new_plugin->argv = malloc(sizeof(char *) * 2); | 
| 128 | if (new_plugin->argv == NULL){ | |
| 24.1.62
by Björn Påhlsson merge + small bugfix | 129 | free(copy_name); | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 130 | free(new_plugin); | 
| 131 | return NULL; | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 132 | } | 
| 24.1.54
by Björn Påhlsson plugin-runner | 133 | new_plugin->argv[0] = copy_name; | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 134 | new_plugin->argv[1] = NULL; | 
| 135
by Teddy Hogeborn * plugin-runner.c (add_environment): Never insert existing environment | 135 |   
 | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 136 | new_plugin->environ = malloc(sizeof(char *)); | 
| 137 | if(new_plugin->environ == NULL){ | |
| 24.1.62
by Björn Påhlsson merge + small bugfix | 138 | free(copy_name); | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 139 | free(new_plugin->argv); | 
| 140 | free(new_plugin); | |
| 141 | return NULL; | |
| 142 | } | |
| 143 | new_plugin->environ[0] = NULL; | |
| 135
by Teddy Hogeborn * plugin-runner.c (add_environment): Never insert existing environment | 144 |   
 | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 145 | /* Append the new plugin to the list */ | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 146 | plugin_list = new_plugin; | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 147 | return new_plugin; | 
| 148 | }
 | |
| 149 | ||
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 150 | /* Helper function for add_argument and add_environment */
 | 
| 151 | static bool add_to_char_array(const char *new, char ***array, | |
| 152 | int *len){ | |
| 153 | /* Resize the pointed-to array to hold one more pointer */ | |
| 154 | *array = realloc(*array, sizeof(char *) | |
| 155 | * (size_t) ((*len) + 2)); | |
| 156 | /* Malloc check */ | |
| 157 | if(*array == NULL){ | |
| 158 | return false; | |
| 159 | } | |
| 160 | /* Make a copy of the new string */ | |
| 161 | char *copy = strdup(new); | |
| 162 | if(copy == NULL){ | |
| 163 | return false; | |
| 164 | } | |
| 165 | /* Insert the copy */ | |
| 166 | (*array)[*len] = copy; | |
| 167 | (*len)++; | |
| 168 | /* Add a new terminating NULL pointer to the last element */ | |
| 169 | (*array)[*len] = NULL; | |
| 170 | return true; | |
| 171 | }
 | |
| 172 | ||
| 173 | /* Add to a plugin's argument vector */
 | |
| 174 | static bool add_argument(plugin *p, const char *arg){ | |
| 175 | if(p == NULL){ | |
| 176 | return false; | |
| 177 | } | |
| 178 | return add_to_char_array(arg, &(p->argv), &(p->argc)); | |
| 179 | }
 | |
| 180 | ||
| 181 | /* Add to a plugin's environment */
 | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 182 | static bool add_environment(plugin *p, const char *def, bool replace){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 183 | if(p == NULL){ | 
| 184 | return false; | |
| 185 | } | |
| 135
by Teddy Hogeborn * plugin-runner.c (add_environment): Never insert existing environment | 186 | /* namelen = length of name of environment variable */ | 
| 187 | size_t namelen = (size_t)(strchrnul(def, '=') - def); | |
| 188 | /* Search for this environment variable */ | |
| 189 | for(char **e = p->environ; *e != NULL; e++){ | |
| 190 | if(strncmp(*e, def, namelen+1) == 0){ | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 191 | /* It already exists */ | 
| 192 | if(replace){ | |
| 193 | char *new = realloc(*e, strlen(def)); | |
| 194 | if(new == NULL){ | |
| 195 | return false; | |
| 196 | } | |
| 197 | *e = new; | |
| 198 | strcpy(*e, def); | |
| 199 | } | |
| 135
by Teddy Hogeborn * plugin-runner.c (add_environment): Never insert existing environment | 200 | return true; | 
| 201 | } | |
| 202 | } | |
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 203 | return add_to_char_array(def, &(p->environ), &(p->envc)); | 
| 204 | }
 | |
| 205 | ||
| 32
by Teddy Hogeborn * plugins.d/plugbasedclient.c (set_cloexec_flag): New function. | 206 | /*
 | 
| 207 |  * Based on the example in the GNU LibC manual chapter 13.13 "File
 | |
| 208 |  * Descriptor Flags".
 | |
| 209 |  * *Note File Descriptor Flags:(libc)Descriptor Flags.
 | |
| 210 |  */
 | |
| 37
by Teddy Hogeborn Non-tested commit for merge purposes. | 211 | static int set_cloexec_flag(int fd) | 
| 32
by Teddy Hogeborn * plugins.d/plugbasedclient.c (set_cloexec_flag): New function. | 212 | {
 | 
| 213 | int ret = fcntl(fd, F_GETFD, 0); | |
| 214 | /* If reading the flags failed, return error indication now. */ | |
| 215 | if(ret < 0){ | |
| 216 | return ret; | |
| 217 | } | |
| 218 | /* Store modified flag word in the descriptor. */ | |
| 219 | return fcntl(fd, F_SETFD, ret | FD_CLOEXEC); | |
| 220 | }
 | |
| 221 | ||
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 222 | |
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 223 | /* Mark processes as completed when they exit, and save their exit
 | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 224 |    status. */
 | 
| 225 | void handle_sigchld(__attribute__((unused)) int sig){ | |
| 24.1.65
by Björn Påhlsson * plugin-runner.c (handle_sigchld): Loop until all exited children | 226 | while(true){ | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 227 | plugin *proc = plugin_list; | 
| 24.1.65
by Björn Påhlsson * plugin-runner.c (handle_sigchld): Loop until all exited children | 228 | int status; | 
| 229 | pid_t pid = waitpid(-1, &status, WNOHANG); | |
| 230 | if(pid == 0){ | |
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 231 | /* Only still running child processes */ | 
| 24.1.65
by Björn Påhlsson * plugin-runner.c (handle_sigchld): Loop until all exited children | 232 | break; | 
| 233 | } | |
| 234 | if(pid == -1){ | |
| 235 | if (errno != ECHILD){ | |
| 236 | perror("waitpid"); | |
| 237 | } | |
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 238 | /* No child processes */ | 
| 239 | break; | |
| 24.1.65
by Björn Påhlsson * plugin-runner.c (handle_sigchld): Loop until all exited children | 240 | } | 
| 241 | ||
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 242 | /* A child exited, find it in process_list */ | 
| 24.1.65
by Björn Påhlsson * plugin-runner.c (handle_sigchld): Loop until all exited children | 243 | while(proc != NULL and proc->pid != pid){ | 
| 244 | proc = proc->next; | |
| 245 | } | |
| 246 | if(proc == NULL){ | |
| 247 | /* Process not found in process list */ | |
| 248 | continue; | |
| 249 | } | |
| 250 | proc->status = status; | |
| 251 | proc->completed = true; | |
| 252 | } | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 253 | }
 | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 254 | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 255 | /* Prints out a password to stdout */
 | 
| 24.1.42
by Björn Påhlsson Added fallback to mandos-client | 256 | bool print_out_password(const char *buffer, size_t length){ | 
| 64
by Teddy Hogeborn * mandos-client.c (print_out_password): Strip trailing '\n'. | 257 | ssize_t ret; | 
| 258 | if(length>0 and buffer[length-1] == '\n'){ | |
| 259 | length--; | |
| 260 | } | |
| 261 | for(size_t written = 0; written < length; written += (size_t)ret){ | |
| 24.1.42
by Björn Påhlsson Added fallback to mandos-client | 262 | ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO, buffer + written, | 
| 263 | length - written)); | |
| 264 | if(ret < 0){ | |
| 265 | return false; | |
| 266 | } | |
| 267 | } | |
| 268 | return true; | |
| 269 | }
 | |
| 270 | ||
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 271 | /* Removes and free a plugin from the plugin list */
 | 
| 272 | static void free_plugin(plugin *plugin_node){ | |
| 273 |   
 | |
| 274 | for(char **arg = plugin_node->argv; *arg != NULL; arg++){ | |
| 275 | free(*arg); | |
| 276 | } | |
| 277 | free(plugin_node->argv); | |
| 278 | for(char **env = plugin_node->environ; *env != NULL; env++){ | |
| 279 | free(*env); | |
| 280 | } | |
| 281 | free(plugin_node->environ); | |
| 282 | free(plugin_node->buffer); | |
| 283 | ||
| 284 | /* Removes the plugin from the singly-linked list */ | |
| 285 | if(plugin_node == plugin_list){ | |
| 286 | /* First one - simple */ | |
| 287 | plugin_list = plugin_list->next; | |
| 288 | } else { | |
| 289 | /* Second one or later */ | |
| 290 | for(plugin *p = plugin_list; p != NULL; p = p->next){ | |
| 291 | if(p->next == plugin_node){ | |
| 292 | p->next = plugin_node->next; | |
| 293 | break; | |
| 294 | } | |
| 295 | } | |
| 296 | } | |
| 297 |   
 | |
| 298 | free(plugin_node); | |
| 299 | }
 | |
| 300 | ||
| 301 | static void free_plugin_list(void){ | |
| 302 | while(plugin_list != NULL){ | |
| 303 | free_plugin(plugin_list); | |
| 98
by Teddy Hogeborn * plugin-runner.c (getplugin): Only copy "name" if not NULL. Free | 304 | } | 
| 24.1.54
by Björn Påhlsson plugin-runner | 305 | }
 | 
| 306 | ||
| 13
by Björn Påhlsson Added following support: | 307 | int main(int argc, char *argv[]){ | 
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 308 | char *plugindir = NULL; | 
| 309 | char *argfile = NULL; | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 310 | FILE *conffp; | 
| 24.1.5
by Björn Påhlsson plugbasedclient: | 311 | size_t d_name_len; | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 312 | DIR *dir = NULL; | 
| 13
by Björn Påhlsson Added following support: | 313 | struct dirent *dirst; | 
| 314 | struct stat st; | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 315 | fd_set rfds_all; | 
| 13
by Björn Påhlsson Added following support: | 316 | int ret, maxfd = 0; | 
| 24.1.6
by Björn Påhlsson plugbasedclient | 317 | uid_t uid = 65534; | 
| 318 | gid_t gid = 65534; | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 319 | bool debug = false; | 
| 320 | int exitstatus = EXIT_SUCCESS; | |
| 24.1.7
by Björn Påhlsson merge | 321 | struct sigaction old_sigchld_action; | 
| 322 | struct sigaction sigchld_action = { .sa_handler = handle_sigchld, | |
| 323 | .sa_flags = SA_NOCLDSTOP }; | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 324 | char **custom_argv = NULL; | 
| 325 | int custom_argc = 0; | |
| 326 |   
 | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 327 | /* Establish a signal handler */ | 
| 328 | sigemptyset(&sigchld_action.sa_mask); | |
| 329 | ret = sigaddset(&sigchld_action.sa_mask, SIGCHLD); | |
| 24.1.54
by Björn Påhlsson plugin-runner | 330 | if(ret == -1){ | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 331 | perror("sigaddset"); | 
| 24.1.54
by Björn Påhlsson plugin-runner | 332 | exitstatus = EXIT_FAILURE; | 
| 333 | goto fallback; | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 334 | } | 
| 335 | ret = sigaction(SIGCHLD, &sigchld_action, &old_sigchld_action); | |
| 24.1.54
by Björn Påhlsson plugin-runner | 336 | if(ret == -1){ | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 337 | perror("sigaction"); | 
| 98
by Teddy Hogeborn * plugin-runner.c (getplugin): Only copy "name" if not NULL. Free | 338 | exitstatus = EXIT_FAILURE; | 
| 24.1.54
by Björn Påhlsson plugin-runner | 339 | goto fallback; | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 340 | } | 
| 341 |   
 | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 342 | /* The options we understand. */ | 
| 343 | struct argp_option options[] = { | |
| 344 | { .name = "global-options", .key = 'g', | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 345 | .arg = "OPTION[,OPTION[,...]]", | 
| 346 | .doc = "Options passed to all plugins" }, | |
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 347 | { .name = "global-env", .key = 'G', | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 348 | .arg = "VAR=value", | 
| 349 | .doc = "Environment variable passed to all plugins" }, | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 350 | { .name = "options-for", .key = 'o', | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 351 | .arg = "PLUGIN:OPTION[,OPTION[,...]]", | 
| 352 | .doc = "Options passed only to specified plugin" }, | |
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 353 | { .name = "env-for", .key = 'E', | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 354 | .arg = "PLUGIN:ENV=value", | 
| 355 | .doc = "Environment variable passed to specified plugin" }, | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 356 | { .name = "disable", .key = 'd', | 
| 357 | .arg = "PLUGIN", | |
| 358 | .doc = "Disable a specific plugin", .group = 1 }, | |
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 359 | { .name = "enable", .key = 'e', | 
| 360 | .arg = "PLUGIN", | |
| 361 | .doc = "Enable a specific plugin", .group = 1 }, | |
| 24.1.5
by Björn Påhlsson plugbasedclient: | 362 | { .name = "plugin-dir", .key = 128, | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 363 | .arg = "DIRECTORY", | 
| 364 | .doc = "Specify a different plugin directory", .group = 2 }, | |
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 365 | { .name = "config-file", .key = 129, | 
| 366 | .arg = "FILE", | |
| 367 | .doc = "Specify a different configuration file", .group = 2 }, | |
| 368 | { .name = "userid", .key = 130, | |
| 369 | .arg = "ID", .flags = 0, | |
| 370 | .doc = "User ID the plugins will run as", .group = 3 }, | |
| 371 | { .name = "groupid", .key = 131, | |
| 372 | .arg = "ID", .flags = 0, | |
| 373 | .doc = "Group ID the plugins will run as", .group = 3 }, | |
| 374 | { .name = "debug", .key = 132, | |
| 375 | .doc = "Debug mode", .group = 4 }, | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 376 | { .name = NULL } | 
| 377 | }; | |
| 378 |   
 | |
| 133
by Teddy Hogeborn * plugin-runner.c: Break a couple of long lines. | 379 | error_t parse_opt (int key, char *arg, __attribute__((unused)) | 
| 380 | struct argp_state *state) { | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 381 | switch (key) { | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 382 | case 'g': /* --global-options */ | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 383 | if (arg != NULL){ | 
| 24.1.50
by Björn Påhlsson changed from using strtok to strsep | 384 | char *p; | 
| 385 | while((p = strsep(&arg, ",")) != NULL){ | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 386 | if(p[0] == '\0'){ | 
| 24.1.50
by Björn Påhlsson changed from using strtok to strsep | 387 | continue; | 
| 388 | } | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 389 | if(not add_argument(getplugin(NULL), p)){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 390 | perror("add_argument"); | 
| 391 | return ARGP_ERR_UNKNOWN; | |
| 392 | } | |
| 393 | } | |
| 394 | } | |
| 395 | break; | |
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 396 | case 'G': /* --global-env */ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 397 | if(arg == NULL){ | 
| 398 | break; | |
| 399 | } | |
| 400 | { | |
| 401 | char *envdef = strdup(arg); | |
| 402 | if(envdef == NULL){ | |
| 403 | break; | |
| 404 | } | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 405 | if(not add_environment(getplugin(NULL), envdef, true)){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 406 | perror("add_environment"); | 
| 24.1.50
by Björn Påhlsson changed from using strtok to strsep | 407 | } | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 408 | } | 
| 409 | break; | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 410 | case 'o': /* --options-for */ | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 411 | if (arg != NULL){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 412 | char *p_name = strsep(&arg, ":"); | 
| 413 | if(p_name[0] == '\0'){ | |
| 24.1.50
by Björn Påhlsson changed from using strtok to strsep | 414 | break; | 
| 415 | } | |
| 416 | char *opt = strsep(&arg, ":"); | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 417 | if(opt[0] == '\0'){ | 
| 24.1.50
by Björn Påhlsson changed from using strtok to strsep | 418 | break; | 
| 419 | } | |
| 420 | if(opt != NULL){ | |
| 421 | char *p; | |
| 422 | while((p = strsep(&opt, ",")) != NULL){ | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 423 | if(p[0] == '\0'){ | 
| 24.1.50
by Björn Påhlsson changed from using strtok to strsep | 424 | continue; | 
| 425 | } | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 426 | if(not add_argument(getplugin(p_name), p)){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 427 | perror("add_argument"); | 
| 428 | return ARGP_ERR_UNKNOWN; | |
| 429 | } | |
| 24.1.50
by Björn Påhlsson changed from using strtok to strsep | 430 | } | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 431 | } | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 432 | } | 
| 433 | break; | |
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 434 | case 'E': /* --env-for */ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 435 | if(arg == NULL){ | 
| 436 | break; | |
| 437 | } | |
| 438 | { | |
| 439 | char *envdef = strchr(arg, ':'); | |
| 440 | if(envdef == NULL){ | |
| 441 | break; | |
| 442 | } | |
| 443 | char *p_name = strndup(arg, (size_t) (envdef-arg)); | |
| 444 | if(p_name == NULL){ | |
| 445 | break; | |
| 446 | } | |
| 447 | envdef++; | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 448 | if(not add_environment(getplugin(p_name), envdef, true)){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 449 | perror("add_environment"); | 
| 450 | } | |
| 451 | } | |
| 452 | break; | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 453 | case 'd': /* --disable */ | 
| 24.1.5
by Björn Påhlsson plugbasedclient: | 454 | if (arg != NULL){ | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 455 | plugin *p = getplugin(arg); | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 456 | if(p == NULL){ | 
| 457 | return ARGP_ERR_UNKNOWN; | |
| 458 | } | |
| 459 | p->disabled = true; | |
| 24.1.5
by Björn Påhlsson plugbasedclient: | 460 | } | 
| 461 | break; | |
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 462 | case 'e': /* --enable */ | 
| 463 | if (arg != NULL){ | |
| 464 | plugin *p = getplugin(arg); | |
| 465 | if(p == NULL){ | |
| 466 | return ARGP_ERR_UNKNOWN; | |
| 467 | } | |
| 468 | p->disabled = false; | |
| 469 | } | |
| 470 | break; | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 471 | case 128: /* --plugin-dir */ | 
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 472 | plugindir = strdup(arg); | 
| 473 | if(plugindir == NULL){ | |
| 474 | perror("strdup"); | |
| 475 | } | |
| 24.1.5
by Björn Påhlsson plugbasedclient: | 476 | break; | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 477 | case 129: /* --config-file */ | 
| 137
by Teddy Hogeborn * plugin-runner.c (main/parse_opt): Removed code for "--config-file". | 478 | /* This is already done by parse_opt_config_file() */ | 
| 479 | break; | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 480 | case 130: /* --userid */ | 
| 24.1.6
by Björn Påhlsson plugbasedclient | 481 | uid = (uid_t)strtol(arg, NULL, 10); | 
| 482 | break; | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 483 | case 131: /* --groupid */ | 
| 24.1.6
by Björn Påhlsson plugbasedclient | 484 | gid = (gid_t)strtol(arg, NULL, 10); | 
| 485 | break; | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 486 | case 132: /* --debug */ | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 487 | debug = true; | 
| 488 | break; | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 489 | case ARGP_KEY_ARG: | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 490 | fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg); | 
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 491 | break; | 
| 492 | case ARGP_KEY_END: | |
| 493 | break; | |
| 494 | default: | |
| 495 | return ARGP_ERR_UNKNOWN; | |
| 496 | } | |
| 497 | return 0; | |
| 498 | } | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 499 |   
 | 
| 137
by Teddy Hogeborn * plugin-runner.c (main/parse_opt): Removed code for "--config-file". | 500 | /* This option parser is the same as parse_opt() above, except it | 
| 501 |      ignores everything but the --config-file option. */
 | |
| 502 | error_t parse_opt_config_file (int key, char *arg, | |
| 503 | __attribute__((unused)) | |
| 504 | struct argp_state *state) { | |
| 505 | switch (key) { | |
| 506 | case 'g': /* --global-options */ | |
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 507 | case 'G': /* --global-env */ | 
| 137
by Teddy Hogeborn * plugin-runner.c (main/parse_opt): Removed code for "--config-file". | 508 | case 'o': /* --options-for */ | 
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 509 | case 'E': /* --env-for */ | 
| 137
by Teddy Hogeborn * plugin-runner.c (main/parse_opt): Removed code for "--config-file". | 510 | case 'd': /* --disable */ | 
| 138
by Teddy Hogeborn * plugin-runner.c: Changed short option for "--global-env" to "-G", | 511 | case 'e': /* --enable */ | 
| 137
by Teddy Hogeborn * plugin-runner.c (main/parse_opt): Removed code for "--config-file". | 512 | case 128: /* --plugin-dir */ | 
| 513 | break; | |
| 514 | case 129: /* --config-file */ | |
| 515 | argfile = strdup(arg); | |
| 516 | if(argfile == NULL){ | |
| 517 | perror("strdup"); | |
| 518 | } | |
| 519 | break; | |
| 520 | case 130: /* --userid */ | |
| 521 | case 131: /* --groupid */ | |
| 522 | case 132: /* --debug */ | |
| 523 | case ARGP_KEY_ARG: | |
| 524 | case ARGP_KEY_END: | |
| 525 | break; | |
| 526 | default: | |
| 527 | return ARGP_ERR_UNKNOWN; | |
| 528 | } | |
| 529 | return 0; | |
| 530 | } | |
| 531 |   
 | |
| 532 | struct argp argp = { .options = options, | |
| 533 | .parser = parse_opt_config_file, | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 534 | .args_doc = "", | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 535 | .doc = "Mandos plugin runner -- Run plugins" }; | 
| 536 |   
 | |
| 137
by Teddy Hogeborn * plugin-runner.c (main/parse_opt): Removed code for "--config-file". | 537 | /* Parse using the parse_opt_config_file in order to get the custom | 
| 538 |      config file location, if any. */
 | |
| 539 | ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL); | |
| 540 | if (ret == ARGP_ERR_UNKNOWN){ | |
| 541 | fprintf(stderr, "Unknown error while parsing arguments\n"); | |
| 542 | exitstatus = EXIT_FAILURE; | |
| 543 | goto fallback; | |
| 544 | } | |
| 545 |   
 | |
| 546 | /* Reset to the normal argument parser */ | |
| 547 | argp.parser = parse_opt; | |
| 548 |   
 | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 549 | /* Open the configfile if available */ | 
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 550 | if (argfile == NULL){ | 
| 551 | conffp = fopen(AFILE, "r"); | |
| 552 | } else { | |
| 553 | conffp = fopen(argfile, "r"); | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 554 | } | 
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 555 | if(conffp != NULL){ | 
| 556 | char *org_line = NULL; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 557 | char *p, *arg, *new_arg, *line; | 
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 558 | size_t size = 0; | 
| 559 | ssize_t sret; | |
| 560 | const char whitespace_delims[] = " \r\t\f\v\n"; | |
| 561 | const char comment_delim[] = "#"; | |
| 562 | ||
| 24.1.67
by Björn Påhlsson transformed a function to a part of main | 563 | custom_argc = 1; | 
| 564 | custom_argv = malloc(sizeof(char*) * 2); | |
| 565 | if(custom_argv == NULL){ | |
| 566 | perror("malloc"); | |
| 567 | exitstatus = EXIT_FAILURE; | |
| 568 | goto fallback; | |
| 569 | } | |
| 570 | custom_argv[0] = argv[0]; | |
| 571 | custom_argv[1] = NULL; | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 572 | |
| 133
by Teddy Hogeborn * plugin-runner.c: Break a couple of long lines. | 573 | /* for each line in the config file, strip whitespace and ignore | 
| 574 |        commented text */
 | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 575 | while(true){ | 
| 576 | sret = getline(&org_line, &size, conffp); | |
| 577 | if(sret == -1){ | |
| 578 | break; | |
| 579 | } | |
| 580 | ||
| 581 | line = org_line; | |
| 582 | arg = strsep(&line, comment_delim); | |
| 583 | while((p = strsep(&arg, whitespace_delims)) != NULL){ | |
| 584 | if(p[0] == '\0'){ | |
| 585 | continue; | |
| 586 | } | |
| 587 | new_arg = strdup(p); | |
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 588 | if(new_arg == NULL){ | 
| 589 | perror("strdup"); | |
| 590 | exitstatus = EXIT_FAILURE; | |
| 591 | free(org_line); | |
| 592 | goto fallback; | |
| 593 | } | |
| 594 | 	
 | |
| 24.1.67
by Björn Påhlsson transformed a function to a part of main | 595 | custom_argc += 1; | 
| 596 | custom_argv = realloc(custom_argv, sizeof(char *) | |
| 597 | * ((unsigned int) custom_argc + 1)); | |
| 598 | if(custom_argv == NULL){ | |
| 599 | perror("realloc"); | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 600 | exitstatus = EXIT_FAILURE; | 
| 24.1.67
by Björn Påhlsson transformed a function to a part of main | 601 | free(org_line); | 
| 24.1.54
by Björn Påhlsson plugin-runner | 602 | goto fallback; | 
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 603 | } | 
| 24.1.67
by Björn Påhlsson transformed a function to a part of main | 604 | custom_argv[custom_argc-1] = new_arg; | 
| 605 | custom_argv[custom_argc] = NULL; | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 606 | } | 
| 607 | } | |
| 608 | free(org_line); | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 609 | } else { | 
| 24.1.52
by Björn Påhlsson merge + minor adjustments | 610 | /* Check for harmful errors and go to fallback. Other errors might | 
| 611 |        not affect opening plugins */
 | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 612 | if (errno == EMFILE or errno == ENFILE or errno == ENOMEM){ | 
| 613 | perror("fopen"); | |
| 24.1.8
by Björn Påhlsson plugbasedclient | 614 | exitstatus = EXIT_FAILURE; | 
| 24.1.54
by Björn Påhlsson plugin-runner | 615 | goto fallback; | 
| 24.1.8
by Björn Påhlsson plugbasedclient | 616 | } | 
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 617 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 618 | /* If there was any arguments from configuration file, | 
| 619 |      pass them to parser as command arguments */
 | |
| 24.1.51
by Björn Påhlsson Added configuration files support for mandos-client | 620 | if(custom_argv != NULL){ | 
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 621 | ret = argp_parse (&argp, custom_argc, custom_argv, ARGP_IN_ORDER, | 
| 622 | 0, NULL); | |
| 24.1.26
by Björn Påhlsson tally count of used symbols | 623 | if (ret == ARGP_ERR_UNKNOWN){ | 
| 64
by Teddy Hogeborn * mandos-client.c (print_out_password): Strip trailing '\n'. | 624 | fprintf(stderr, "Unknown error while parsing arguments\n"); | 
| 24.1.26
by Björn Påhlsson tally count of used symbols | 625 | exitstatus = EXIT_FAILURE; | 
| 24.1.54
by Björn Påhlsson plugin-runner | 626 | goto fallback; | 
| 24.1.26
by Björn Påhlsson tally count of used symbols | 627 | } | 
| 24.1.8
by Björn Påhlsson plugbasedclient | 628 | } | 
| 629 |   
 | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 630 | /* Parse actual command line arguments, to let them override the | 
| 631 |      config file */
 | |
| 632 | ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL); | |
| 633 | if (ret == ARGP_ERR_UNKNOWN){ | |
| 634 | fprintf(stderr, "Unknown error while parsing arguments\n"); | |
| 635 | exitstatus = EXIT_FAILURE; | |
| 636 | goto fallback; | |
| 637 | } | |
| 638 |   
 | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 639 | if(debug){ | 
| 640 | for(plugin *p = plugin_list; p != NULL; p=p->next){ | |
| 641 | fprintf(stderr, "Plugin: %s has %d arguments\n", | |
| 32
by Teddy Hogeborn * plugins.d/plugbasedclient.c (set_cloexec_flag): New function. | 642 | p->name ? p->name : "Global", p->argc - 1); | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 643 | for(char **a = p->argv; *a != NULL; a++){ | 
| 644 | fprintf(stderr, "\tArg: %s\n", *a); | |
| 645 | } | |
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 646 | fprintf(stderr, "...and %u environment variables\n", p->envc); | 
| 647 | for(char **a = p->environ; *a != NULL; a++){ | |
| 648 | fprintf(stderr, "\t%s\n", *a); | |
| 649 | } | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 650 | } | 
| 651 | } | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 652 |   
 | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 653 | /* Strip permissions down to nobody */ | 
| 24.1.6
by Björn Påhlsson plugbasedclient | 654 | ret = setuid(uid); | 
| 655 | if (ret == -1){ | |
| 656 | perror("setuid"); | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 657 | } | 
| 24.1.6
by Björn Påhlsson plugbasedclient | 658 | setgid(gid); | 
| 659 | if (ret == -1){ | |
| 39
by Teddy Hogeborn * plugins.d/mandosclient.c (pgp_packet_decrypt): Renamed variables. | 660 | perror("setgid"); | 
| 24.1.6
by Björn Påhlsson plugbasedclient | 661 | } | 
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 662 |   
 | 
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 663 | if (plugindir == NULL){ | 
| 664 | dir = opendir(PDIR); | |
| 665 | } else { | |
| 666 | dir = opendir(plugindir); | |
| 667 | } | |
| 24.1.6
by Björn Påhlsson plugbasedclient | 668 |   
 | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 669 | if(dir == NULL){ | 
| 670 | perror("Could not open plugin dir"); | |
| 671 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 672 | goto fallback; | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 673 | } | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 674 |   
 | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 675 | /* Set the FD_CLOEXEC flag on the directory, if possible */ | 
| 676 | { | |
| 677 | int dir_fd = dirfd(dir); | |
| 678 | if(dir_fd >= 0){ | |
| 679 | ret = set_cloexec_flag(dir_fd); | |
| 680 | if(ret < 0){ | |
| 681 | perror("set_cloexec_flag"); | |
| 682 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 683 | goto fallback; | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 684 | } | 
| 685 | } | |
| 686 | } | |
| 24.1.1
by Björn Påhlsson Added syntax and support for plugbasedclient arguments and how they | 687 |   
 | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 688 | FD_ZERO(&rfds_all); | 
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 689 |   
 | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 690 | /* Read and execute any executable in the plugin directory*/ | 
| 13
by Björn Påhlsson Added following support: | 691 | while(true){ | 
| 692 | dirst = readdir(dir); | |
| 693 |     
 | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 694 | /* All directory entries have been processed */ | 
| 13
by Björn Påhlsson Added following support: | 695 | if(dirst == NULL){ | 
| 24.1.26
by Björn Påhlsson tally count of used symbols | 696 | if (errno == EBADF){ | 
| 697 | perror("readdir"); | |
| 698 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 699 | goto fallback; | 
| 24.1.26
by Björn Påhlsson tally count of used symbols | 700 | } | 
| 13
by Björn Påhlsson Added following support: | 701 | break; | 
| 702 | } | |
| 703 |     
 | |
| 704 | d_name_len = strlen(dirst->d_name); | |
| 705 |     
 | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 706 | /* Ignore dotfiles, backup files and other junk */ | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 707 | { | 
| 708 | bool bad_name = false; | |
| 709 |       
 | |
| 710 | const char const *bad_prefixes[] = { ".", "#", NULL }; | |
| 711 |       
 | |
| 712 | const char const *bad_suffixes[] = { "~", "#", ".dpkg-new", | |
| 713 | ".dpkg-old", | |
| 714 | ".dpkg-divert", NULL }; | |
| 715 | for(const char **pre = bad_prefixes; *pre != NULL; pre++){ | |
| 716 | size_t pre_len = strlen(*pre); | |
| 717 | if((d_name_len >= pre_len) | |
| 718 | and strncmp((dirst->d_name), *pre, pre_len) == 0){ | |
| 719 | if(debug){ | |
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 720 | fprintf(stderr, "Ignoring plugin dir entry \"%s\"" | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 721 | " with bad prefix %s\n", dirst->d_name, *pre); | 
| 722 | } | |
| 723 | bad_name = true; | |
| 724 | break; | |
| 725 | } | |
| 726 | } | |
| 727 | if(bad_name){ | |
| 728 | continue; | |
| 729 | } | |
| 730 | for(const char **suf = bad_suffixes; *suf != NULL; suf++){ | |
| 731 | size_t suf_len = strlen(*suf); | |
| 732 | if((d_name_len >= suf_len) | |
| 733 | and (strcmp((dirst->d_name)+d_name_len-suf_len, *suf) | |
| 734 | == 0)){ | |
| 735 | if(debug){ | |
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 736 | fprintf(stderr, "Ignoring plugin dir entry \"%s\"" | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 737 | " with bad suffix %s\n", dirst->d_name, *suf); | 
| 738 | } | |
| 739 | bad_name = true; | |
| 740 | break; | |
| 741 | } | |
| 742 | } | |
| 743 |       
 | |
| 744 | if(bad_name){ | |
| 745 | continue; | |
| 746 | } | |
| 13
by Björn Påhlsson Added following support: | 747 | } | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 748 | |
| 749 | char *filename; | |
| 750 | ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name); | |
| 751 | if(ret < 0){ | |
| 752 | perror("asprintf"); | |
| 24.1.46
by Björn Påhlsson mandos-client | 753 | continue; | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 754 | } | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 755 |     
 | 
| 24.1.26
by Björn Påhlsson tally count of used symbols | 756 | ret = stat(filename, &st); | 
| 757 | if (ret == -1){ | |
| 758 | perror("stat"); | |
| 24.1.46
by Björn Påhlsson mandos-client | 759 | free(filename); | 
| 760 | continue; | |
| 24.1.26
by Björn Påhlsson tally count of used symbols | 761 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 762 | |
| 763 | /* Ignore non-executable files */ | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 764 | if (not S_ISREG(st.st_mode) or (access(filename, X_OK) != 0)){ | 
| 765 | if(debug){ | |
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 766 | fprintf(stderr, "Ignoring plugin dir entry \"%s\"" | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 767 | " with bad type or mode\n", filename); | 
| 768 | } | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 769 | free(filename); | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 770 | continue; | 
| 771 | } | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 772 |     
 | 
| 773 | plugin *p = getplugin(dirst->d_name); | |
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 774 | if(p == NULL){ | 
| 775 | perror("getplugin"); | |
| 776 | free(filename); | |
| 777 | continue; | |
| 778 | } | |
| 779 | if(p->disabled){ | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 780 | if(debug){ | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 781 | fprintf(stderr, "Ignoring disabled plugin \"%s\"\n", | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 782 | dirst->d_name); | 
| 783 | } | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 784 | free(filename); | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 785 | continue; | 
| 786 | } | |
| 32
by Teddy Hogeborn * plugins.d/plugbasedclient.c (set_cloexec_flag): New function. | 787 | { | 
| 788 | /* Add global arguments to argument list for this plugin */ | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 789 | plugin *g = getplugin(NULL); | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 790 | if(g != NULL){ | 
| 791 | for(char **a = g->argv + 1; *a != NULL; a++){ | |
| 792 | if(not add_argument(p, *a)){ | |
| 793 | perror("add_argument"); | |
| 794 | } | |
| 795 | } | |
| 796 | /* Add global environment variables */ | |
| 797 | for(char **e = g->environ; *e != NULL; e++){ | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 798 | if(not add_environment(p, *e, false)){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 799 | perror("add_environment"); | 
| 800 | } | |
| 801 | } | |
| 802 | } | |
| 803 | } | |
| 804 | /* If this plugin has any environment variables, we will call | |
| 805 |        using execve and need to duplicate the environment from this
 | |
| 806 |        process, too. */
 | |
| 807 | if(p->environ[0] != NULL){ | |
| 808 | for(char **e = environ; *e != NULL; e++){ | |
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 809 | if(not add_environment(p, *e, false)){ | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 810 | perror("add_environment"); | 
| 811 | } | |
| 812 | } | |
| 813 | } | |
| 814 |     
 | |
| 98
by Teddy Hogeborn * plugin-runner.c (getplugin): Only copy "name" if not NULL. Free | 815 | int pipefd[2]; | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 816 | ret = pipe(pipefd); | 
| 817 | if (ret == -1){ | |
| 818 | perror("pipe"); | |
| 819 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 820 | goto fallback; | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 821 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 822 | /* Ask OS to automatic close the pipe on exec */ | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 823 | ret = set_cloexec_flag(pipefd[0]); | 
| 824 | if(ret < 0){ | |
| 825 | perror("set_cloexec_flag"); | |
| 826 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 827 | goto fallback; | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 828 | } | 
| 829 | ret = set_cloexec_flag(pipefd[1]); | |
| 830 | if(ret < 0){ | |
| 831 | perror("set_cloexec_flag"); | |
| 832 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 833 | goto fallback; | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 834 | } | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 835 | /* Block SIGCHLD until process is safely in process list */ | 
| 836 | ret = sigprocmask (SIG_BLOCK, &sigchld_action.sa_mask, NULL); | |
| 837 | if(ret < 0){ | |
| 838 | perror("sigprocmask"); | |
| 839 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 840 | goto fallback; | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 841 | } | 
| 136
by Teddy Hogeborn * plugin-runner.c (add_environment): Override existing environment | 842 | /* Starting a new process to be watched */ | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 843 | pid_t pid = fork(); | 
| 24.1.46
by Björn Påhlsson mandos-client | 844 | if(pid == -1){ | 
| 845 | perror("fork"); | |
| 846 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 847 | goto fallback; | 
| 24.1.46
by Björn Påhlsson mandos-client | 848 | } | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 849 | if(pid == 0){ | 
| 850 | /* this is the child process */ | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 851 | ret = sigaction(SIGCHLD, &old_sigchld_action, NULL); | 
| 852 | if(ret < 0){ | |
| 853 | perror("sigaction"); | |
| 854 | _exit(EXIT_FAILURE); | |
| 855 | } | |
| 856 | ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL); | |
| 857 | if(ret < 0){ | |
| 858 | perror("sigprocmask"); | |
| 859 | _exit(EXIT_FAILURE); | |
| 860 | } | |
| 24.1.26
by Björn Påhlsson tally count of used symbols | 861 | |
| 862 | ret = dup2(pipefd[1], STDOUT_FILENO); /* replace our stdout */ | |
| 863 | if(ret == -1){ | |
| 864 | perror("dup2"); | |
| 865 | _exit(EXIT_FAILURE); | |
| 866 | } | |
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 867 |       
 | 
| 868 | if(dirfd(dir) < 0){ | |
| 869 | /* If dir has no file descriptor, we could not set FD_CLOEXEC | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 870 | 	   above and must now close it manually here. */
 | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 871 | closedir(dir); | 
| 33
by Teddy Hogeborn * plugins.d/plugbasedclient.c (main): Close the pipe fd after dup2:ing | 872 | } | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 873 | if(p->environ[0] == NULL){ | 
| 874 | if(execv(filename, p->argv) < 0){ | |
| 875 | perror("execv"); | |
| 876 | _exit(EXIT_FAILURE); | |
| 877 | } | |
| 878 | } else { | |
| 879 | if(execve(filename, p->argv, p->environ) < 0){ | |
| 880 | perror("execve"); | |
| 881 | _exit(EXIT_FAILURE); | |
| 882 | } | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 883 | } | 
| 884 | /* no return */ | |
| 885 | } | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 886 | /* Parent process */ | 
| 887 | close(pipefd[1]); /* Close unused write end of pipe */ | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 888 | free(filename); | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 889 | plugin *new_plugin = getplugin(dirst->d_name); | 
| 890 | if (new_plugin == NULL){ | |
| 891 | perror("getplugin"); | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 892 | ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL); | 
| 893 | if(ret < 0){ | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 894 | perror("sigprocmask"); | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 895 | } | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 896 | exitstatus = EXIT_FAILURE; | 
| 24.1.54
by Björn Påhlsson plugin-runner | 897 | goto fallback; | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 898 | } | 
| 899 |     
 | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 900 | new_plugin->pid = pid; | 
| 901 | new_plugin->fd = pipefd[0]; | |
| 902 |     
 | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 903 | /* Unblock SIGCHLD so signal handler can be run if this process | 
| 904 |        has already completed */
 | |
| 905 | ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL); | |
| 906 | if(ret < 0){ | |
| 907 | perror("sigprocmask"); | |
| 908 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 909 | goto fallback; | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 910 | } | 
| 911 |     
 | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 912 | FD_SET(new_plugin->fd, &rfds_all); | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 913 |     
 | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 914 | if (maxfd < new_plugin->fd){ | 
| 915 | maxfd = new_plugin->fd; | |
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 916 | } | 
| 917 |     
 | |
| 13
by Björn Påhlsson Added following support: | 918 | } | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 919 |   
 | 
| 13
by Björn Påhlsson Added following support: | 920 | closedir(dir); | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 921 | dir = NULL; | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 922 | |
| 923 | for(plugin *p = plugin_list; p != NULL; p = p->next){ | |
| 924 | if(p->pid != 0){ | |
| 925 | break; | |
| 926 | } | |
| 927 | if(p->next == NULL){ | |
| 928 | fprintf(stderr, "No plugin processes started. Incorrect plugin" | |
| 929 | " directory?\n"); | |
| 930 | free_plugin_list(); | |
| 931 | } | |
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 932 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 933 | |
| 934 | /* Main loop while running plugins exist */ | |
| 935 | while(plugin_list){ | |
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 936 | fd_set rfds = rfds_all; | 
| 937 | int select_ret = select(maxfd+1, &rfds, NULL, NULL, NULL); | |
| 938 | if (select_ret == -1){ | |
| 939 | perror("select"); | |
| 940 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 941 | goto fallback; | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 942 | } | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 943 | /* OK, now either a process completed, or something can be read | 
| 944 |        from one of them */
 | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 945 | for(plugin *proc = plugin_list; proc != NULL; proc = proc->next){ | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 946 | /* Is this process completely done? */ | 
| 947 | if(proc->eof and proc->completed){ | |
| 948 | /* Only accept the plugin output if it exited cleanly */ | |
| 949 | if(not WIFEXITED(proc->status) | |
| 950 | or WEXITSTATUS(proc->status) != 0){ | |
| 951 | /* Bad exit by plugin */ | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 952 | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 953 | if(debug){ | 
| 954 | if(WIFEXITED(proc->status)){ | |
| 60
by Teddy Hogeborn * mandos-client.c (main): Cast pid_t to unsigned int before printing. | 955 | fprintf(stderr, "Plugin %u exited with status %d\n", | 
| 956 | (unsigned int) (proc->pid), | |
| 957 | WEXITSTATUS(proc->status)); | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 958 | } else if(WIFSIGNALED(proc->status)) { | 
| 60
by Teddy Hogeborn * mandos-client.c (main): Cast pid_t to unsigned int before printing. | 959 | fprintf(stderr, "Plugin %u killed by signal %d\n", | 
| 960 | (unsigned int) (proc->pid), | |
| 961 | WTERMSIG(proc->status)); | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 962 | } else if(WCOREDUMP(proc->status)){ | 
| 60
by Teddy Hogeborn * mandos-client.c (main): Cast pid_t to unsigned int before printing. | 963 | fprintf(stderr, "Plugin %d dumped core\n", | 
| 964 | (unsigned int) (proc->pid)); | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 965 | } | 
| 966 | } | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 967 | 	  
 | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 968 | /* Remove the plugin */ | 
| 969 | FD_CLR(proc->fd, &rfds_all); | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 970 | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 971 | /* Block signal while modifying process_list */ | 
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 972 | ret = sigprocmask(SIG_BLOCK, &sigchld_action.sa_mask, NULL); | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 973 | if(ret < 0){ | 
| 974 | perror("sigprocmask"); | |
| 975 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 976 | goto fallback; | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 977 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 978 | free_plugin(proc); | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 979 | /* We are done modifying process list, so unblock signal */ | 
| 980 | ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, | |
| 981 | NULL); | |
| 982 | if(ret < 0){ | |
| 983 | perror("sigprocmask"); | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 984 | exitstatus = EXIT_FAILURE; | 
| 985 | goto fallback; | |
| 986 | } | |
| 987 | 	  
 | |
| 988 | if(plugin_list == NULL){ | |
| 989 | break; | |
| 990 | } | |
| 991 | continue; | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 992 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 993 | 	
 | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 994 | /* This process exited nicely, so print its buffer */ | 
| 24.1.42
by Björn Påhlsson Added fallback to mandos-client | 995 | |
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 996 | bool bret = print_out_password(proc->buffer, | 
| 997 | proc->buffer_length); | |
| 24.1.42
by Björn Påhlsson Added fallback to mandos-client | 998 | if(not bret){ | 
| 999 | perror("print_out_password"); | |
| 1000 | exitstatus = EXIT_FAILURE; | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1001 | } | 
| 24.1.54
by Björn Påhlsson plugin-runner | 1002 | goto fallback; | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1003 | } | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 1004 |       
 | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1005 | /* This process has not completed. Does it have any output? */ | 
| 1006 | if(proc->eof or not FD_ISSET(proc->fd, &rfds)){ | |
| 1007 | /* This process had nothing to say at this time */ | |
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 1008 | continue; | 
| 1009 | } | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1010 | /* Before reading, make the process' data buffer large enough */ | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 1011 | if(proc->buffer_length + BUFFER_SIZE > proc->buffer_size){ | 
| 1012 | proc->buffer = realloc(proc->buffer, proc->buffer_size | |
| 1013 | + (size_t) BUFFER_SIZE); | |
| 1014 | if (proc->buffer == NULL){ | |
| 1015 | perror("malloc"); | |
| 1016 | exitstatus = EXIT_FAILURE; | |
| 24.1.54
by Björn Påhlsson plugin-runner | 1017 | goto fallback; | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 1018 | } | 
| 1019 | proc->buffer_size += BUFFER_SIZE; | |
| 1020 | } | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1021 | /* Read from the process */ | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 1022 | ret = read(proc->fd, proc->buffer + proc->buffer_length, | 
| 1023 | BUFFER_SIZE); | |
| 1024 | if(ret < 0){ | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1025 | /* Read error from this process; ignore the error */ | 
| 34
by Teddy Hogeborn * plugbasedclient.c (main): Check if plugin dir could be opened. Set | 1026 | continue; | 
| 1027 | } | |
| 1028 | if(ret == 0){ | |
| 1029 | /* got EOF */ | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1030 | proc->eof = true; | 
| 1031 | } else { | |
| 1032 | proc->buffer_length += (size_t) ret; | |
| 13
by Björn Påhlsson Added following support: | 1033 | } | 
| 1034 | } | |
| 1035 | } | |
| 24.1.45
by Björn Påhlsson Fixed fallback on error in mandos-client | 1036 | |
| 1037 | ||
| 24.1.54
by Björn Påhlsson plugin-runner | 1038 | fallback: | 
| 24.1.42
by Björn Påhlsson Added fallback to mandos-client | 1039 |   
 | 
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 1040 | if(plugin_list == NULL or exitstatus != EXIT_SUCCESS){ | 
| 102
by Teddy Hogeborn * plugin-runner.c (process): Set "completed" and "status" as volatile. | 1041 | /* Fallback if all plugins failed, none are found or an error | 
| 1042 |        occured */
 | |
| 24.1.42
by Björn Påhlsson Added fallback to mandos-client | 1043 | bool bret; | 
| 1044 | fprintf(stderr, "Going to fallback mode using getpass(3)\n"); | |
| 1045 | char *passwordbuffer = getpass("Password: "); | |
| 1046 | bret = print_out_password(passwordbuffer, strlen(passwordbuffer)); | |
| 1047 | if(not bret){ | |
| 1048 | perror("print_out_password"); | |
| 1049 | exitstatus = EXIT_FAILURE; | |
| 1050 | } | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1051 | } | 
| 24.1.42
by Björn Påhlsson Added fallback to mandos-client | 1052 |   
 | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1053 | /* Restore old signal handler */ | 
| 24.1.54
by Björn Påhlsson plugin-runner | 1054 | ret = sigaction(SIGCHLD, &old_sigchld_action, NULL); | 
| 1055 | if(ret == -1){ | |
| 1056 | perror("sigaction"); | |
| 1057 | exitstatus = EXIT_FAILURE; | |
| 1058 | } | |
| 1059 | ||
| 1060 | if(custom_argv != NULL){ | |
| 24.1.67
by Björn Påhlsson transformed a function to a part of main | 1061 | for(char **arg = custom_argv+1; *arg != NULL; arg++){ | 
| 24.1.54
by Björn Påhlsson plugin-runner | 1062 | free(*arg); | 
| 78
by Teddy Hogeborn Add feature to specify custom environment variables for plugins. | 1063 | } | 
| 24.1.54
by Björn Påhlsson plugin-runner | 1064 | free(custom_argv); | 
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1065 | } | 
| 1066 |   
 | |
| 1067 | if(dir != NULL){ | |
| 1068 | closedir(dir); | |
| 1069 | } | |
| 1070 |   
 | |
| 1071 | /* Free the process list and kill the processes */ | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 1072 | for(plugin *p = plugin_list; p != NULL; p = p->next){ | 
| 1073 | if(p->pid != 0){ | |
| 1074 | close(p->fd); | |
| 1075 | ret = kill(p->pid, SIGTERM); | |
| 1076 | if(ret == -1 and errno != ESRCH){ | |
| 1077 | /* Set-uid proccesses might not get closed */ | |
| 1078 | perror("kill"); | |
| 1079 | } | |
| 24.1.26
by Björn Påhlsson tally count of used symbols | 1080 | } | 
| 13
by Björn Påhlsson Added following support: | 1081 | } | 
| 1082 |   
 | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1083 | /* Wait for any remaining child processes to terminate */ | 
| 1084 | do{ | |
| 1085 | ret = wait(NULL); | |
| 1086 | } while(ret >= 0); | |
| 1087 | if(errno != ECHILD){ | |
| 1088 | perror("wait"); | |
| 1089 | } | |
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 1090 | |
| 24.1.76
by Björn Påhlsson Rearranged so plugins and processes is the same thing | 1091 | free_plugin_list(); | 
| 1092 |   
 | |
| 24.1.69
by Björn Påhlsson added configfile as a optional argument to plugin-runner | 1093 | free(plugindir); | 
| 1094 | free(argfile); | |
| 35
by Teddy Hogeborn * plugbasedclient.c (struct process): New fields "eof", "completed", | 1095 |   
 | 
| 31
by Teddy Hogeborn * plugins.d/plugbasedclient.c: Update include file comments. | 1096 | return exitstatus; | 
| 13
by Björn Påhlsson Added following support: | 1097 | }
 |