From: Luiz Augusto von Dentz <[email protected]>
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.2
------------------------------------------------------------------------------
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. Data protection magic?
Nope - It's vRanger. Get your free trial download today.
http://p.sf.net/sfu/quest-sfdev2dev
_______________________________________________
Openobex-users mailing list
[email protected]
http://lists.sourceforge.net/lists/listinfo/openobex-users
