|
 |
e1e6be |
diff -up pnm2ppa-1.04/calibrate_ppa.c.coverity-return-local pnm2ppa-1.04/calibrate_ppa.c
|
|
|
e1e6be |
--- pnm2ppa-1.04/calibrate_ppa.c.coverity-return-local 2000-11-11 22:22:38.000000000 +0000
|
|
|
e1e6be |
+++ pnm2ppa-1.04/calibrate_ppa.c 2011-05-20 16:04:23.233425268 +0100
|
|
|
e1e6be |
@@ -75,7 +75,7 @@ main (int argc, char *argv[])
|
|
|
e1e6be |
BOOLEAN gamma = false ;
|
|
|
e1e6be |
FILE *outFile = NULL ;
|
|
|
e1e6be |
int testPattern = 0;
|
|
|
e1e6be |
- char *out_filepath = NULL;
|
|
|
e1e6be |
+ char out_filepath[MAXPATHLEN];
|
|
|
e1e6be |
char c;
|
|
|
e1e6be |
static struct option long_opts[] = {
|
|
|
e1e6be |
{"gamma",0,0,'g'},
|
|
|
e1e6be |
@@ -157,9 +157,7 @@ main (int argc, char *argv[])
|
|
|
e1e6be |
{
|
|
|
e1e6be |
if ( strlen(optarg) < MAXPATHLEN )
|
|
|
e1e6be |
{
|
|
|
e1e6be |
- char filepath[MAXPATHLEN];
|
|
|
e1e6be |
- strncpy(filepath, optarg, MAXPATHLEN);
|
|
|
e1e6be |
- out_filepath = filepath ;
|
|
|
e1e6be |
+ strncpy(out_filepath, optarg, MAXPATHLEN);
|
|
|
e1e6be |
use_stdout = false ;
|
|
|
e1e6be |
}
|
|
|
e1e6be |
}
|
|
|
e1e6be |
diff -up pnm2ppa-1.04/pnm2ppa.c.coverity-return-local pnm2ppa-1.04/pnm2ppa.c
|
|
|
e1e6be |
--- pnm2ppa-1.04/pnm2ppa.c.coverity-return-local 2000-11-11 19:53:03.000000000 +0000
|
|
|
e1e6be |
+++ pnm2ppa-1.04/pnm2ppa.c 2011-05-20 16:04:23.234425246 +0100
|
|
|
e1e6be |
@@ -145,20 +145,15 @@ print_pnm (FILE * fptr)
|
|
|
e1e6be |
char *
|
|
|
e1e6be |
readPath(char *input )
|
|
|
e1e6be |
{
|
|
|
e1e6be |
- char File_path[MAXPATHLEN];
|
|
|
e1e6be |
char *path = NULL;
|
|
|
e1e6be |
/* security audit ajv 20001007; be very careful with input strings!*/
|
|
|
e1e6be |
if (strlen(input) < MAXPATHLEN )
|
|
|
e1e6be |
{
|
|
|
e1e6be |
- if ( strncpy (File_path, input, MAXPATHLEN ))
|
|
|
e1e6be |
- {
|
|
|
e1e6be |
- /*
|
|
|
e1e6be |
- a user-specifed file *File_path may later be opened;
|
|
|
e1e6be |
- should any extra validation of this be performed now ?
|
|
|
e1e6be |
- */
|
|
|
e1e6be |
- path = File_path;
|
|
|
e1e6be |
- return path;
|
|
|
e1e6be |
- }
|
|
|
e1e6be |
+ path = strdup(input);
|
|
|
e1e6be |
+ /*
|
|
|
e1e6be |
+ a user-specifed file *path may later be opened;
|
|
|
e1e6be |
+ should any extra validation of this be performed now ?
|
|
|
e1e6be |
+ */
|
|
|
e1e6be |
}
|
|
|
e1e6be |
else
|
|
|
e1e6be |
{
|