From: Luiz Augusto von Dentz <luiz.von.de...@intel.com> obex_test_client.c: In function ‘push_client’: obex_test_client.c:229:6: error: variable ‘num’ set but not used obex_test_client.c: In function ‘put_client’: obex_test_client.c:290:6: error: variable ‘num’ set but not used obex_test_client.c: In function ‘get_client’: obex_test_client.c:347:6: error: variable ‘num’ set but not used obex_test_client.c: In function ‘setpath_client’: obex_test_client.c:418:6: error: variable ‘num’ set but not used cc1: all warnings being treated as errors --- apps/obex_test/obex_test_client.c | 31 +++++++++++++++++++++++++++---- 1 files changed, 27 insertions(+), 4 deletions(-)
diff --git a/apps/obex_test/obex_test_client.c b/apps/obex_test/obex_test_client.c index 0bf5d42..739f04a 100644 --- a/apps/obex_test/obex_test_client.c +++ b/apps/obex_test/obex_test_client.c @@ -234,7 +234,13 @@ void push_client(obex_t *handle) int file_size; printf("PUSH filename> "); - num = scanf("%s", fname); + memset(fname, 0, sizeof(fname)); + num = scanf("%199c", fname); + if (num != 1) { + perror("scanf"); + return; + } + bfname = strdup(basename(fname)); buf = easy_readfile(fname, &file_size); @@ -293,7 +299,14 @@ void put_client(obex_t *handle) int file_size; printf("PUT file (local, remote)> "); - num = scanf("%s %s", lname, rname); + memset(lname, 0, sizeof(lname)); + memset(rname, 0, sizeof(rname)); + num = scanf("%199c %199c", lname, rname); + if (num != 2) { + perror("scanf:"); + return; + } + buf = easy_readfile(lname, &file_size); if(buf == NULL) { @@ -348,7 +361,12 @@ void get_client(obex_t *handle, struct context *gt) obex_headerdata_t hd; printf("GET File> "); - num = scanf("%s", req_name); + memset(req_name, 0, sizeof(req_name)); + num = scanf("%199c", req_name); + if (num != 1) { + perror("scanf:"); + return; + } if(! (object = OBEX_ObjectNew(handle, OBEX_CMD_GET))) { printf("Error\n"); @@ -419,7 +437,12 @@ void setpath_client(obex_t *handle) obex_headerdata_t hd; printf("SETPATH> "); - num = scanf("%s", path); + memset(path, 0, sizeof(path)); + num = scanf("%199c", path); + if (num != 1) { + perror("scanf:"); + return; + } if(! (object = OBEX_ObjectNew(handle, OBEX_CMD_SETPATH))) { printf("Error\n"); -- 1.7.5.3 ------------------------------------------------------------------------------ Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Discover what all the cheering's about. Get your free trial download today. http://p.sf.net/sfu/quest-dev2dev2 _______________________________________________ Openobex-users mailing list Openobex-users@lists.sourceforge.net http://lists.sourceforge.net/lists/listinfo/openobex-users