Merge pull request #45 from Guenael/guenael

fix(issue #42): Possible resource leaks

rtlsdr_wsprd.c

@@ -445,12 +445,14 @@ int32_t readfile(float *iSamples, float *qSamples, char *filename) {
     int32_t res = fseek(fd, 26, SEEK_SET);
     if (res) {
         fprintf(stderr, "Cannot set file offset...\n");
+        fclose(fd);
         return 1;
     }
 
     int32_t nread = fread(filebuffer, sizeof(float), 2*SIGNAL_LENGHT*SIGNAL_SAMPLE_RATE, fd);
     if (nread != 2*SIGNAL_LENGHT*SIGNAL_SAMPLE_RATE) {
         fprintf(stderr, "Cannot read all the data!\n");
+        fclose(fd);
         return 1;
     }
 

wsprd/wsprd.c

@@ -486,11 +486,9 @@ int32_t wspr_decode(float *idat, float *qdat, uint32_t npoints,
                 sscanf(line,"%d %s",&nh,hcall);
                 strcpy(hashtab+nh*13,hcall);
             }
-        } else {
-            fhash=fopen("hashtable.txt","w+");
-        }
         fclose(fhash);
         }
+    }
 
     // Main loop starts here
     for (int32_t ipass=0; ipass<options.npasses; ipass++) {

wsprd/wsprsim_utils.c

@@ -120,7 +120,7 @@ void pack_prefix(char *callsign, int32_t *n, int32_t *m, int32_t *nadd ) {
         *m=60000 + 26 + *m;
     } else {
         char* pfx=strtok(callsign,"/");
-        call6=strtok(NULL," ");
+        call6=strtok(pfx," ");  // FIXME-CHECK
         *n=pack_call(call6);
         int plen=strlen(pfx);
         if( plen ==1 ) {