/mandos/trunk

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/trunk

« back to all changes in this revision

Viewing changes to plugins.d/usplash.c

  • Committer: Teddy Hogeborn
  • Date: 2008-10-03 09:32:30 UTC
  • Revision ID: teddy@fukt.bsnet.se-20081003093230-rshn19e0c19zz12i
* .bzrignore (plugins.d/askpass-fifo): Added.

* Makefile (FORTIFY): Added "-fstack-protector-all".
  (mandos, mandos-keygen): Use more strict regexps when updating the
                           version number.

* mandos (Client.__init__): Use os.path.expandvars() and
                            os.path.expanduser() on the "secfile"
                            config value.

* plugins.d/splashy.c: Update comments and order of #include's.
  (main): Check user and group when looking for running splashy
          process.  Do not ignore ENOENT from execl().  Use _exit()
          instead of "return" when an error happens in child
          processes.  Bug fix: Only wait for splashy_update
          completion if it was started.  Bug fix: detect failing
          waitpid().  Only kill splashy_update if it is running.  Do
          the killing of the old splashy process before the fork().
          Do setsid() and setuid(geteuid()) before starting the new
          splashy.  Report failing execl().

* plugins.d/usplash.c: Update comments and order of #include's.
  (main): Check user and group when looking for running usplash
          process.  Do not report execv() error if interrupted by a
          signal.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/usplash.c
1
 
/*  -*- coding: utf-8 -*- */
2
 
/*
3
 
 * Usplash - Read a password from usplash and output it
4
 
 * 
5
 
 * Copyright © 2008,2009 Teddy Hogeborn
6
 
 * Copyright © 2008,2009 Björn Påhlsson
7
 
 * 
8
 
 * This program is free software: you can redistribute it and/or
9
 
 * modify it under the terms of the GNU General Public License as
10
 
 * published by the Free Software Foundation, either version 3 of the
11
 
 * License, or (at your option) any later version.
12
 
 * 
13
 
 * This program is distributed in the hope that it will be useful, but
14
 
 * WITHOUT ANY WARRANTY; without even the implied warranty of
15
 
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
16
 
 * General Public License for more details.
17
 
 * 
18
 
 * You should have received a copy of the GNU General Public License
19
 
 * along with this program.  If not, see
20
 
 * <http://www.gnu.org/licenses/>.
21
 
 * 
22
 
 * Contact the authors at <https://www.fukt.bsnet.se/~belorn/> and
23
 
 * <https://www.fukt.bsnet.se/~teddy/>.
24
 
 */
25
 
 
26
1
#define _GNU_SOURCE             /* asprintf() */
27
2
#include <signal.h>             /* sig_atomic_t, struct sigaction,
28
3
                                   sigemptyset(), sigaddset(), SIGINT,
36
11
                                   dirent */
37
12
#include <stddef.h>             /* NULL */
38
13
#include <string.h>             /* strlen(), memcmp() */
39
 
#include <stdio.h>              /* asprintf(), perror(), sscanf() */
 
14
#include <stdio.h>              /* asprintf(), perror() */
40
15
#include <unistd.h>             /* close(), write(), readlink(),
41
16
                                   read(), STDOUT_FILENO, sleep(),
42
17
                                   fork(), setuid(), geteuid(),
43
18
                                   setsid(), chdir(), dup2(),
44
19
                                   STDERR_FILENO, execv() */
45
 
#include <stdlib.h>             /* free(), EXIT_FAILURE, realloc(),
46
 
                                   EXIT_SUCCESS, malloc(), _exit() */
 
20
#include <stdlib.h>             /* free(), EXIT_FAILURE, strtoul(),
 
21
                                   realloc(), EXIT_SUCCESS, malloc(),
 
22
                                   _exit() */
47
23
#include <stdlib.h>             /* getenv() */
48
24
#include <dirent.h>             /* opendir(), readdir(), closedir() */
49
25
#include <sys/stat.h>           /* struct stat, lstat(), S_ISLNK */
91
67
  }
92
68
  
93
69
  size_t written = 0;
94
 
  ssize_t sret = 0;
95
70
  while(not interrupted_by_signal and written < cmd_line_len){
96
 
    sret = write(fifo_fd, cmd_line + written,
97
 
                 cmd_line_len - written);
98
 
    if(sret == -1){
 
71
    ret = write(fifo_fd, cmd_line + written,
 
72
                cmd_line_len - written);
 
73
    if(ret == -1){
99
74
      if(errno != EINTR or interrupted_by_signal){
100
75
        int e = errno;
101
76
        close(fifo_fd);
106
81
        continue;
107
82
      }
108
83
    }
109
 
    written += (size_t)sret;
 
84
    written += (size_t)ret;
110
85
  }
111
86
  free(cmd_line_alloc);
112
87
  do{
169
144
    for(struct dirent *proc_ent = readdir(proc_dir);
170
145
        proc_ent != NULL;
171
146
        proc_ent = readdir(proc_dir)){
172
 
      pid_t pid;
173
 
      /* In the GNU C library, pid_t is always int */
174
 
      ret = sscanf(proc_ent->d_name, "%d", &pid);
175
 
      if(ret != 1){
 
147
      pid_t pid = (pid_t) strtoul(proc_ent->d_name, NULL, 10);
 
148
      if(pid == 0){
176
149
        /* Not a process */
177
150
        continue;
178
151
      }
194
167
        struct stat exe_stat;
195
168
        ret = lstat(exe_link, &exe_stat);
196
169
        if(ret == -1){
197
 
          if(errno == ENOENT){
198
 
            free(exe_link);
199
 
            continue;
200
 
          }
201
170
          perror("lstat");
202
171
          free(exe_link);
203
172
          free(prompt);
213
182
        
214
183
        sret = readlink(exe_link, exe_target, sizeof(exe_target));
215
184
        free(exe_link);
 
185
        if(sret == -1){
 
186
          continue;
 
187
        }
216
188
      }
217
189
      if((sret == ((ssize_t)sizeof(exe_target)-1))
218
190
         and (memcmp(usplash_name, exe_target,
487
459
    /* Child; will become new usplash process */
488
460
    
489
461
    /* Make the effective user ID (root) the only user ID instead of
490
 
       the real user ID (_mandos) */
 
462
       the real user ID (mandos) */
491
463
    ret = setuid(geteuid());
492
464
    if(ret == -1){
493
465
      perror("setuid");