coverity: ftell returns a signed value

The check for flen < 0 could never have been true since flen was declared
to be size_t (unsigned). Declare flen to be long since that is what ftell
returns.

Signed-off-by: Dwight Engen <dwight.engen@oracle.com>
Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

diff --git a/src/lxc/lxccontainer.c b/src/lxc/lxccontainer.c
index 73c347d466074dd94d88b8c138dd3844f8040d19..04a920826dc651471b1f89065787dd274f5da419 100644 (file)
--- a/src/lxc/lxccontainer.c
+++ b/src/lxc/lxccontainer.c
@@ -1133,7 +1133,7 @@ static int update_name_and_paths(const char *path, struct lxc_container *oldc,
                const char *newname, const char *newpath)
 {
        FILE *f;
-       size_t flen;
+       long flen;
        char *contents;
        const char *p0, *p1, *p2, *end;
        const char *oldpath = oldc->get_config_path(oldc);