diff options
| author | Kamil Rytarowski <n54@gmx.com> | 2013-05-11 17:06:45 +0200 |
|---|---|---|
| committer | Harald Hoyer <harald@redhat.com> | 2013-05-28 14:34:29 +0200 |
| commit | 599182b10870a826360091246f0a5f26c691f690 (patch) | |
| tree | 952ae521bb5865f937b874cd25517d39abbcbd73 | |
| parent | 6a736cc129ed29157a4e5460382d17154f8fc381 (diff) | |
| download | dracut-599182b10870a826360091246f0a5f26c691f690.tar.gz dracut-599182b10870a826360091246f0a5f26c691f690.tar.bz2 dracut-599182b10870a826360091246f0a5f26c691f690.zip | |
Introduce stricter type correctness
| -rw-r--r-- | install/dracut-install.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/install/dracut-install.c b/install/dracut-install.c index 584a30b5..33fad4a4 100644 --- a/install/dracut-install.c +++ b/install/dracut-install.c @@ -80,9 +80,9 @@ static char *convert_abs_rel(const char *from, const char *target) _cleanup_free_ char *realtarget = NULL; _cleanup_free_ char *target_dir_p = NULL, *realpath_p = NULL; const char *realfrom = from; - int level = 0, fromlevel = 0, targetlevel = 0; - int l, i, rl; - int dirlen; + size_t level = 0, fromlevel = 0, targetlevel = 0; + int l; + size_t i, rl, dirlen; target_dir_p = strdup(target); if (!target_dir_p) @@ -215,7 +215,7 @@ static int cp(const char *src, const char *dst) if (ret == 0) { struct timeval tv[2]; if (fchown(dest_desc, sb.st_uid, sb.st_gid) != 0) - fchown(dest_desc, -1, sb.st_gid); + fchown(dest_desc, (__uid_t)-1, sb.st_gid); tv[0].tv_sec = sb.st_atime; tv[0].tv_usec = 0; tv[1].tv_sec = sb.st_mtime; @@ -297,7 +297,7 @@ static int resolve_deps(const char *src) log_debug("ldd: '%s'", buf); if (strstr(buf, "you do not have execution permission")) { - log_error(buf); + log_error("%s", buf); ret+=1; break; } @@ -670,7 +670,7 @@ static int parse_argv(int argc, char *argv[]) static int resolve_lazy(int argc, char **argv) { int i; - int destrootdirlen = strlen(destrootdir); + size_t destrootdirlen = strlen(destrootdir); int ret = 0; char *item; for (i = 0; i < argc; i++) { |