Fixes:

plug/hotplug.c:152:10: error: ignoring return value of 'chown', declared with 
attribute warn_unused_result [-Werror=unused-result]
     chown(blobmsg_get_string(tb[0]), 0, g->gr_gid);

Signed-off-by: Florian Fainelli <f.faine...@gmail.com>
---
 plug/hotplug.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/plug/hotplug.c b/plug/hotplug.c
index 85959155634f..54a90cc92f2d 100644
--- a/plug/hotplug.c
+++ b/plug/hotplug.c
@@ -130,6 +130,7 @@ static void handle_makedev(struct blob_attr *msg, struct 
blob_attr *data)
        char *minor = hotplug_msg_find_var(msg, "MINOR");
        char *major = hotplug_msg_find_var(msg, "MAJOR");
        char *subsystem = hotplug_msg_find_var(msg, "SUBSYSTEM");
+       int ret;
 
        blobmsg_parse_array(mkdev_policy, 3, tb, blobmsg_data(data), 
blobmsg_data_len(data));
        if (tb[0] && tb[1] && minor && major && subsystem) {
@@ -149,8 +150,9 @@ static void handle_makedev(struct blob_attr *msg, struct 
blob_attr *data)
                        struct group *g = getgrnam(blobmsg_get_string(tb[2]));
 
                        if (g)
-                               chown(blobmsg_get_string(tb[0]), 0, g->gr_gid);
-                       else
+                               ret = chown(blobmsg_get_string(tb[0]), 0, 
g->gr_gid);
+
+                       if (!g || ret < 0)
                                ERROR("cannot set group %s for %s\n",
                                        blobmsg_get_string(tb[2]),
                                        blobmsg_get_string(tb[0]));
-- 
2.9.3


_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev

Reply via email to