app: actually resolve the interpreter through extension as last resort.

Current code of gimp_interpreter_db_resolve() was only resolving the
interpreter by the file extension when the file could not be opened for
reading or if it was empty/coult not be read. This basically made this
test completely useless.
Let's fix this. Now it will be run all the time, but simply at the end,
if shebang and magic failed.

(cherry picked from commit 8509117fe6)
This commit is contained in:
Jehan 2018-06-16 01:16:45 +02:00
parent 1516bfc14b
commit 51a8ff9c37
1 changed files with 23 additions and 15 deletions

View File

@ -752,7 +752,7 @@ resolve_magic (GimpInterpreterDB *db,
list = list->next;
}
return resolve_extension (db, program_path);
return NULL;
}
gchar *
@ -762,8 +762,7 @@ gimp_interpreter_db_resolve (GimpInterpreterDB *db,
{
GFile *file;
GInputStream *input;
gsize bytes_read;
gchar buffer[BUFSIZE];
gchar *program = NULL;
g_return_val_if_fail (GIMP_IS_INTERPRETER_DB (db), NULL);
g_return_val_if_fail (program_path != NULL, NULL);
@ -775,22 +774,31 @@ gimp_interpreter_db_resolve (GimpInterpreterDB *db,
input = G_INPUT_STREAM (g_file_read (file, NULL, NULL));
g_object_unref (file);
if (! input)
return resolve_extension (db, program_path);
if (input)
{
gsize bytes_read;
gchar buffer[BUFSIZE];
memset (buffer, 0, sizeof (buffer));
g_input_stream_read_all (input, buffer,
sizeof (buffer) - 1, /* leave one nul at the end */
&bytes_read, NULL, NULL);
g_object_unref (input);
memset (buffer, 0, sizeof (buffer));
g_input_stream_read_all (input, buffer,
sizeof (buffer) - 1, /* leave one nul at the end */
&bytes_read, NULL, NULL);
g_object_unref (input);
if (bytes_read == 0)
return resolve_extension (db, program_path);
if (bytes_read)
{
if (bytes_read > 3 && buffer[0] == '#' && buffer[1] == '!')
program = resolve_sh_bang (db, program_path, buffer, bytes_read, interp_arg);
if (bytes_read > 3 && buffer[0] == '#' && buffer[1] == '!')
return resolve_sh_bang (db, program_path, buffer, bytes_read, interp_arg);
if (! program)
program = resolve_magic (db, program_path, buffer);
}
}
return resolve_magic (db, program_path, buffer);
if (! program)
program = resolve_extension (db, program_path);
return program;
}
static void