diff options
| author | kay.sievers@vrfy.org <kay.sievers@vrfy.org> | 2004-02-26 19:40:22 -0800 |
|---|---|---|
| committer | Greg KH <gregkh@suse.de> | 2005-04-26 21:32:30 -0700 |
| commit | 831f800da34ddb449aecae925cd1f154f20b1eed (patch) | |
| tree | e58de05592252bab5a83fca240941be313cba94b /udev_config.c | |
| parent | bef370d6ebd6707cc2ef183c2dc83f4a62d8111b (diff) | |
[PATCH] udev - safer string handling - part two
As promised, here is the next round. We provide in addition to the
already used macros:
strfieldcpy(to, from)
strfieldcat(to, from)
the corresponding friends, if the size of the target is not known and
must be provided by the caller:
strnfieldcpy(to, from, maxsize)
strnfieldcat(to, from, maxsize)
and switch nearly all possibly unsafe users of strcat(), strncat(),
strcpy() and strncpy() to these safer macros.
The last known remaining issue seems the use of sprintf() and
snprintf(). I will take on it later today or tomorrow.
Diffstat (limited to 'udev_config.c')
| -rw-r--r-- | udev_config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/udev_config.c b/udev_config.c index cade81cf20..de83ef9c4c 100644 --- a/udev_config.c +++ b/udev_config.c @@ -81,7 +81,7 @@ static void init_variables(void) #define set_var(_name, _var) \ if (strcasecmp(variable, _name) == 0) { \ dbg_parse("%s = '%s'", _name, value); \ - strncpy(_var, value, sizeof(_var)); \ + strnfieldcpy(_var, value, sizeof(_var));\ } #define set_bool(_name, _var) \ |