Chih-Wei Huang  committed 7929deb

Fix compiling warnings

The patch removes the following warnings:

system/vold/vdc.c: In function 'do_cmd':
system/vold/vdc.c:71:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

In file included from system/vold/Fat.cpp:34:0:
bionic/libc/kernel/common/linux/fs.h:109:0: warning: "MS_RMT_MASK" redefined [enabled by default]
In file included from system/vold/Fat.cpp:31:0:
bionic/libc/include/sys/mount.h:64:0: note: this is the location of the previous definition

Change-Id: I1e950dcec87f8bae0cec61a1611126b3abc0c90e

  • Participants
  • Parent commits b1596af

Comments (0)

Files changed (5)

File Devmapper.cpp

     io->version[2] = 0;
     io->flags = flags;
     if (name) {
-        int ret = strlcpy(io->name, name, sizeof(io->name));
+        size_t ret = strlcpy(io->name, name, sizeof(io->name));
 	if (ret >= sizeof(io->name))
 #include <sys/mount.h>
 #include <linux/kdev_t.h>
-#include <linux/fs.h>
 #define LOG_TAG "Vold"
 #include <sys/mount.h>
 #include <linux/kdev_t.h>
-#include <linux/fs.h>
 #define LOG_TAG "Vold"
 #include <sys/param.h>
 #include <linux/kdev_t.h>
-#include <linux/fs.h>
 #include <cutils/properties.h>
 static int do_cmd(int sock, int argc, char **argv) {
     char final_cmd[255] = "0 "; /* 0 is a (now required) sequence number */
     int i;
-    int ret;
+    size_t ret;
     for (i = 1; i < argc; i++) {
         char *cmp;