uuidd: use pid_t type when referring to process id

Earlier use of a variable that holds switch enabling boolean to hold process
id was a little bit strange, and not exactly correct.  An int should be good
enough to hold any pid, but it is better to be precise and use the type that
is meant for the job.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
This commit is contained in:
Sami Kerola 2020-11-20 22:05:46 +00:00
parent 1e416dc0b9
commit c14e0f6239
No known key found for this signature in database
GPG Key ID: 0D46FEF7E61DBB46
1 changed files with 8 additions and 6 deletions

View File

@ -586,7 +586,7 @@ int main(int argc, char **argv)
uuidd_cxt.debug = 1;
break;
case 'k':
do_kill++;
do_kill = 1;
break;
case 'n':
num = strtou32_or_err(optarg,
@ -694,17 +694,19 @@ int main(int argc, char **argv)
if (do_kill) {
ret = call_daemon(socket_path, UUIDD_OP_GETPID, buf, sizeof(buf), 0, NULL);
if ((ret > 0) && ((do_kill = atoi((char *) buf)) > 0)) {
ret = kill(do_kill, SIGTERM);
if (0 < ret) {
pid_t pid;
pid = (pid_t)strtou32_or_err(buf, _("failed to parse pid"));
ret = kill(pid, SIGTERM);
if (ret < 0) {
if (!uuidd_cxt.quiet)
warn(_("couldn't kill uuidd running "
"at pid %d"), do_kill);
"at pid %d"), pid);
return EXIT_FAILURE;
}
if (!uuidd_cxt.quiet)
printf(_("Killed uuidd running at pid %d.\n"),
do_kill);
printf(_("Killed uuidd running at pid %d.\n"), pid);
}
return EXIT_SUCCESS;
}