core
/
eccodes
mirror of https://github.com/ecmwf/eccodes.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ECC-539: grib_copy/grib_set - suggestion to avoid overwriting files (Part 1)

This commit is contained in:
Shahram Najm 2017-09-22 09:58:21 +01:00
parent 529d7c3a4c
commit 61209e2ddb
1 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
tools/grib_tools.c
View File

@ -14,6 +14,8 @@
*/
#include "grib_tools.h"
#include <stdlib.h>
#if HAVE_LIBJASPER
/* Remove compiler warnings re macros being redefined */
#undef PACKAGE_BUGREPORT
@ -982,6 +984,16 @@ void grib_print_full_statistics(grib_runtime_options* options)
options->filter_handle_count,options->handle_count,options->file_count);
}
static int filenames_equal(const char * f1, const char * f2)
{
char resolved1[8192] = {0,};
char resolved2[8192] = {0,};
realpath(f1, resolved1);
realpath(f2, resolved2);
int eq = (strcmp(resolved1, resolved2)==0);
return eq;
}
void grib_tools_write_message(grib_runtime_options* options, grib_handle* h)
{
const void *buffer;
@ -1001,6 +1013,13 @@ void grib_tools_write_message(grib_runtime_options* options, grib_handle* h)
err = grib_recompose_name(h, NULL, options->outfile->name, filename, 0);
// Check outfile is not same as infile
if (filenames_equal(options->infile->name, filename)) {
grib_context_log(h->context, GRIB_LOG_ERROR,
"output file '%s' is the same as input file. Aborting\n", filename);
exit(GRIB_IO_PROBLEM);
}
of = grib_file_open(filename, "w", &err);
if (!of || !of->handle) {