annotate compat/getline.c @ 33:fa93d2ff9c62

Prevent potential division by zero Add safeguard ensuring that there is at least one possible character to generate passwords from in order to exclude the possibility of a division by zero error in rand_uniform.
author Guido Berhoerster <guido+pwm@berhoerster.name>
date Tue, 30 Jul 2019 20:38:08 +0200
parents 5bdea77d0c1d
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
26
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
1 /*
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
2 * Copyright (C) 2016 Guido Berhoerster <guido+pwm@berhoerster.name>
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
3 *
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
4 * Permission is hereby granted, free of charge, to any person obtaining
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
5 * a copy of this software and associated documentation files (the
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
6 * "Software"), to deal in the Software without restriction, including
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
7 * without limitation the rights to use, copy, modify, merge, publish,
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
8 * distribute, sublicense, and/or sell copies of the Software, and to
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
9 * permit persons to whom the Software is furnished to do so, subject to
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
10 * the following conditions:
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
11 *
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
12 * The above copyright notice and this permission notice shall be included
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
13 * in all copies or substantial portions of the Software.
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
14 *
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
15 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
16 * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
17 * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
18 * IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
19 * CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
20 * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
21 * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
22 */
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
23
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
24 #include <stdio.h>
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
25 #include <stdlib.h>
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
26 #include <limits.h>
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
27 #include <sys/types.h>
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
28 #include <errno.h>
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
29
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
30 #include "getline.h"
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
31
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
32 #define MIN(x, y) (((x) < (y)) ? (x) : (y))
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
33
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
34 ssize_t
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
35 getdelim(char **linep, size_t *sizep, int delim, FILE *fp)
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
36 {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
37 ssize_t len;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
38 char *line = *linep;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
39 size_t size = *sizep;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
40 size_t pos = 0;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
41 char *line_new;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
42 int c;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
43
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
44 if (*linep == NULL) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
45 size = 0;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
46 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
47
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
48 flockfile(fp);
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
49
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
50 for (;;) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
51 if (size - pos < 2) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
52 size = MIN((size_t)SSIZE_MAX + 1,
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
53 (size < BUFSIZ) ? BUFSIZ : size * 1.5);
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
54 if (size - pos < 2) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
55 len = -1;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
56 errno = EOVERFLOW;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
57 break;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
58 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
59 line_new = realloc(line, size);
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
60 if (line_new == NULL) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
61 len = -1;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
62 break;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
63 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
64 *linep = line = line_new;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
65 *sizep = size;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
66 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
67 c = fgetc(fp);
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
68 if (c == EOF) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
69 len = -1;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
70 break;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
71 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
72 line[pos] = c;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
73 pos++;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
74 len = pos;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
75 if (c == delim) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
76 break;
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
77 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
78 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
79
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
80 if (*linep != NULL) {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
81 (*linep)[pos] = '\0';
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
82 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
83
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
84 funlockfile(fp);
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
85
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
86 return (len);
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
87 }
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
88
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
89 ssize_t
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
90 getline(char **linep, size_t *sizep, FILE *fp)
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
91 {
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
92 return (getdelim(linep, sizep, '\n', fp));
5bdea77d0c1d Add pwm-clip utility for setting the X11 CLIPBOARD selection
Guido Berhoerster <guido+pwm@berhoerster.name>
parents:
diff changeset
93 }