Changeset 4339f09 in mainline for uspace/app/bdsh/cmds/modules/rm/rm.c

Timestamp:

2013-06-28T07:46:31Z (12 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

e52b4b5

Parents:

3de67b4c (diff), dba3e2c (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Mostly C style (obey clang suggestions)

Merge from lp:~vojtech-horky/helenos/misc.

Most changes driven by warnings/errors from clang, such as type
redefinitions (typedef struct …) or possibly uninitizalied
variables. Clang still emits some warnings but a lot less than
before . As a result, kernel can be built with clang/-Werror.

Also updated mkext2.py to work with Python 3 again.

File:

• uspace/app/bdsh/cmds/modules/rm/rm.c (modified) (2 diffs)

uspace/app/bdsh/cmds/modules/rm/rm.c

@@ -99 +99 @@
         if (NULL == (rm->nwd = (char *) malloc(PATH_MAX)))
                 return 0;
-        memset(rm->nwd, 0, sizeof(rm->nwd));
+        memset(rm->nwd, 0, PATH_MAX);
 
         if (NULL == (rm->owd = (char *) malloc(PATH_MAX)))
                 return 0;
-        memset(rm->owd, 0, sizeof(rm->owd));
+        memset(rm->owd, 0, PATH_MAX);
 
         if (NULL == (rm->cwd = (char *) malloc(PATH_MAX)))
                 return 0;
-        memset(rm->cwd, 0, sizeof(rm->cwd));
+        memset(rm->cwd, 0, PATH_MAX);
 
         chdir(".");
@@ -298 +298 @@
                         break;
                 }
-                memset(buff, 0, sizeof(buff));
+                memset(buff, 0, len);
                 snprintf(buff, len, "%s", argv[i]);