pv 968773, author Chris Pascoe <c.pascoe@itee.uq.edu.au>, rv vapo - minor fixes for...
authorVlad Apostolov <vapo@sgi.com>
Wed, 8 Aug 2007 07:54:05 +0000 (07:54 +0000)
committerVlad Apostolov <vapo@sgi.com>
Wed, 8 Aug 2007 07:54:05 +0000 (07:54 +0000)
Merge of master-melb:xfs-cmds:29348a by kenmcd.

  pv 968773, author Chris Pascoe <c.pascoe@itee.uq.edu.au>, rv vapo - minor fixes for XFS DMAPI tests

dmapi/src/common/lib/util.c
dmapi/src/simple/dm_getall_tokens.c
dmapi/src/suite1/cmd/print_event.c
dmapi/src/suite2/src/dm_test_daemon.c

index 0160b384cbfde4f84f74a5968504f2ce11776539..958b7e456059b1f5f6787b4687431b8418c56038 100644 (file)
@@ -311,11 +311,11 @@ setup_dmapi(dm_sessid_t    *sidp)
        char    *cp;
 
        if (dm_init_service(&cp) == -1)  {
-               err_msg("%s/%d: Can't init dmapi", __FILE__, __LINE__);
+               err_msg("%s/%d: Can't init dmapi\n", __FILE__, __LINE__);
                return(1);
        }
        if (strcmp(cp, DM_VER_STR_CONTENTS)) {
-               err_msg("%s/%d: Compiled for a different version", __FILE__, __LINE__);
+               err_msg("%s/%d: Compiled for a different version\n", __FILE__, __LINE__);
                return(1);
        }
 
@@ -372,7 +372,7 @@ save_filedata(
        retval = 0;
        filebuf = malloc(CHUNKSIZE);
        if (filebuf == NULL) {
-               err_msg("%s/%d: Can't alloc memory for file buffer", __FILE__, __LINE__);
+               err_msg("%s/%d: Can't alloc memory for file buffer\n", __FILE__, __LINE__);
                goto out;
        }
 
@@ -476,7 +476,7 @@ restore_filedata(
 
        filebuf = malloc(CHUNKSIZE);
        if (filebuf == NULL) {
-               err_msg("%s/%d: Can't alloc memory for file buffer", __FILE__, __LINE__);
+               err_msg("%s/%d: Can't alloc memory for file buffer\n", __FILE__, __LINE__);
                goto out;
        }
 
index bb15b7e8f41c9d26ea95f56cf5f2038ef29b6449..2df4f018eded9fe7f4b5422740453a44215cba74 100644 (file)
@@ -73,7 +73,7 @@ main( int argc, char **argv )
 
        printf("tokens=\"");
        for( i = 0; i < rnelem; i++ ){
-               printf("%d ", (int)(tokenbuf+i));
+               printf("%d ", (int)*(tokenbuf+i));
        }
        printf("\"\n");
        exit(0);
index c926e05f4b970708af0427ec0d6366f61e68c046..9a4beba9af544010530ddb6c1406e442cb4908ce 100644 (file)
@@ -1151,7 +1151,7 @@ finish_responding(
 {
   int          error = 0;
   u_int                nbytes, ntokens = 0, ret_ntokens, i;
-  dm_token_t   *tokenbuf = NULL;
+  dm_token_t   *tokenbuf = NULL, *tokenptr;
   size_t       buflen, ret_buflen;
   char         *msgbuf = NULL;
   dm_eventmsg_t        *msg;
@@ -1173,14 +1173,16 @@ finish_responding(
    * If we're already using the returned size, double it and try again.
    */
   do {
+    dm_token_t *tmpbuf;
     ntokens = (ntokens != ret_ntokens) ? ret_ntokens : ntokens*2;
     nbytes = ntokens * (sizeof(dm_token_t) + sizeof(dm_vardata_t));
-    tokenbuf = malloc(nbytes);
-    if (tokenbuf == NULL) {
+    tmpbuf = realloc(tokenbuf, nbytes);
+    if (tmpbuf == NULL) {
       err_msg("Can't malloc %d bytes for tokenbuf\n", nbytes);
       error = 1;
       goto out;
     }
+    tokenbuf = tmpbuf;
     error = dm_getall_tokens(sid, ntokens, tokenbuf, &ret_ntokens);
   } while (error && errno == E2BIG);
 
@@ -1189,25 +1191,28 @@ finish_responding(
     goto out;
   }
 
+  tokenptr = tokenbuf;
   for (i = 0; i < ret_ntokens; i++) {
     if (Verbose)
-      err_msg("Responding to outstanding event for token %d\n",(int)*tokenbuf);
+      err_msg("Responding to outstanding event for token %d\n",(int)*tokenptr);
 
     /*
      * The E2BIG dance reprise...
      */
     do {
+      char *tmpbuf;
       buflen = (buflen != ret_buflen) ? ret_buflen : buflen * 2;
-      msgbuf = malloc(buflen);
-      if (msgbuf == NULL) {
+      tmpbuf = realloc(msgbuf, buflen);
+      if (tmpbuf == NULL) {
        err_msg("Can't malloc %d bytes for msgbuf\n", buflen);
        error = 1;
        goto out;
       }
-      error = dm_find_eventmsg(sid, *tokenbuf, buflen, msgbuf, &ret_buflen);
+      msgbuf = tmpbuf;
+      error = dm_find_eventmsg(sid, *tokenptr, buflen, msgbuf, &ret_buflen);
     } while (error && errno == E2BIG);
     if (error) {
-      errno_msg("Can't find the event message for token %d", (int)*tokenbuf);
+      errno_msg("Can't find the event message for token %d", (int)*tokenptr);
       goto out;
     }
 
@@ -1219,7 +1224,7 @@ finish_responding(
       msg = DM_STEP_TO_NEXT(msg, dm_eventmsg_t *);
     }
 
-    tokenbuf++;
+    tokenptr++;
   }
 
  out:
index e47cad2a699f1e7d03c0e1d70071fb898c88647f..214750aa1f5e3e26cde9e176fec735bb28cce921 100644 (file)
@@ -1171,7 +1171,7 @@ finish_responding(
 {
   int          error = 0;
   u_int                nbytes, ntokens = 0, ret_ntokens, i;
-  dm_token_t   *tokenbuf = NULL;
+  dm_token_t   *tokenbuf = NULL, *tokenptr;
   size_t       buflen, ret_buflen;
   char         *msgbuf = NULL;
   dm_eventmsg_t        *msg;
@@ -1193,14 +1193,16 @@ finish_responding(
    * If we're already using the returned size, double it and try again.
    */
   do {
+    dm_token_t *tmpbuf;
     ntokens = (ntokens != ret_ntokens) ? ret_ntokens : ntokens*2;
     nbytes = ntokens * (sizeof(dm_token_t) + sizeof(dm_vardata_t));
-    tokenbuf = malloc(nbytes);
-    if (tokenbuf == NULL) {
+    tmpbuf = realloc(tokenbuf, nbytes);
+    if (tmpbuf == NULL) {
       err_msg("Can't malloc %d bytes for tokenbuf\n", nbytes);
       error = 1;
       goto out;
     }
+    tokenbuf = tmpbuf;
     error = dm_getall_tokens(sid, ntokens, tokenbuf, &ret_ntokens);
   } while (error && errno == E2BIG);
 
@@ -1209,25 +1211,28 @@ finish_responding(
     goto out;
   }
 
+  tokenptr = tokenbuf;
   for (i = 0; i < ret_ntokens; i++) {
     if (Verbose)
-      err_msg("Responding to outstanding event for token %d\n",(int)*tokenbuf);
+      err_msg("Responding to outstanding event for token %d\n",(int)*tokenptr);
 
     /*
      * The E2BIG dance reprise...
      */
     do {
+      char *tmpbuf;
       buflen = (buflen != ret_buflen) ? ret_buflen : buflen * 2;
-      msgbuf = malloc(buflen);
-      if (msgbuf == NULL) {
+      tmpbuf = realloc(msgbuf, buflen);
+      if (tmpbuf == NULL) {
        err_msg("Can't malloc %d bytes for msgbuf\n", buflen);
        error = 1;
        goto out;
       }
-      error = dm_find_eventmsg(sid, *tokenbuf, buflen, msgbuf, &ret_buflen);
+      msgbuf = tmpbuf;
+      error = dm_find_eventmsg(sid, *tokenptr, buflen, msgbuf, &ret_buflen);
     } while (error && errno == E2BIG);
     if (error) {
-      errno_msg("Can't find the event message for token %d", (int)*tokenbuf);
+      errno_msg("Can't find the event message for token %d", (int)*tokenptr);
       goto out;
     }
 
@@ -1239,7 +1244,7 @@ finish_responding(
       msg = DM_STEP_TO_NEXT(msg, dm_eventmsg_t *);
     }
 
-    tokenbuf++;
+    tokenptr++;
   }
 
  out: