dtcalc: resolve coverity issues
authorPeter Howkins <flibble@users.sf.net>
Fri, 30 Mar 2018 19:24:04 +0000 (20:24 +0100)
committerPeter Howkins <flibble@users.sf.net>
Fri, 30 Mar 2018 19:24:04 +0000 (20:24 +0100)
cde/programs/dtcalc/ds_xlib.c
cde/programs/dtcalc/motif.c

index 2e13a0d21897242727197d6ab769e654d98fa9e1..a9489804487f9b9dbd09a2e59c4ebfe2d9119efa 100644 (file)
 #include <Xm/Xm.h>
 #include <Xm/Protocols.h>
 
-#define  FREE         (void) free
-#define  FPRINTF      (void) fprintf
-#define  GETHOSTNAME  (void) gethostname
-#define  SPRINTF      (void) sprintf
-#define  STRCPY       (void) strcpy
 
 #define  EQUAL(a, b)  !strncmp(a, b, strlen(b))
 #define  MAXLINE      120        /* Maximum length for character strings. */
@@ -153,7 +148,7 @@ ds_load_resources(Display *display)
     db = XrmGetStringDatabase(XResourceManagerString(display)) ;
   else
     { 
-      SPRINTF(name, "%s/.Xdefaults", home) ;
+      snprintf(name, sizeof(name), "%s/.Xdefaults", home) ;
       db = XrmGetFileDatabase(name) ;
     }
   XrmMergeDatabases(db, &rDB) ;
@@ -166,7 +161,7 @@ ds_load_resources(Display *display)
     {
       snprintf(name, sizeof(name), "%s/.Xdefaults-", home) ;
       len = strlen(name) ;
-      GETHOSTNAME(name+len, 1024-len) ;
+      gethostname(name+len, 1024-len) ;
       db = XrmGetFileDatabase(name) ;
     }
   else db = XrmGetFileDatabase(ptr) ;
@@ -267,7 +262,7 @@ ds_save_resources(XrmDatabase rDB, char *filename)
       {
         home = getenv("HOME") ;
         filename = (char*) calloc(1, strlen(home) + 18) ;
-        SPRINTF(filename, "%s/.dtcalcdef", home) ;
+        snprintf(filename, sizeof(filename), "%s/.dtcalcdef", home) ;
       }
   }
 
@@ -275,14 +270,14 @@ ds_save_resources(XrmDatabase rDB, char *filename)
 
   if (stat(filename, &statbuf) != -1 && access(filename, W_OK) != 0)
     { 
-      FREE(filename) ;
+      free(filename) ;
       return(1) ;
     }
 
 /* If file does not exist this call will create it. */
 
   XrmPutFileDatabase(rDB, filename) ;
-  FREE(filename) ;
+  free(filename) ;
   return(0) ;
 }
 
index 042af244e0b44935b272551c4e1f9aaa1822661e..e828c6651750ba8a20b7cf5aba79fcc26c0a2e1e 100644 (file)
@@ -1984,8 +1984,8 @@ get_next_event(Widget widget, int ev_action, XEvent *xevent)
       else if (ksym == XK_Meta_R  && down) return(META);
       else if (ksym == XK_Meta_R  && up) return(META);
       else if (ksym == XK_Alt_L  && down) return(ALT);
-      else if (ksym == XK_Alt_R  && up) return(ALT);
-      else if (ksym == XK_Alt_L  && down) return(ALT);
+      else if (ksym == XK_Alt_L  && up) return(ALT);
+      else if (ksym == XK_Alt_R  && down) return(ALT);
       else if (ksym == XK_Alt_R  && up) return(ALT);
       else if (ksym == XK_Select  && down) return(SPACE);
       else if (ksym == XK_Select  && up) return(SPACE);