Skip to content

Commit

Permalink
fuzz: add datadir for oss-fuzz compatability
Browse files Browse the repository at this point in the history
This allows us to keep pc-bios in executable_dir/pc-bios, rather than
executable_dir/../pc-bios, which is incompatible with oss-fuzz' file
structure.

Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
Reviewed-by: Darren Kenny <darren.kenny@oracle.com>
Message-id: 20200512030133.29896-2-alxndr@bu.edu
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
  • Loading branch information
a1xndr authored and stefanhaRH committed Jun 5, 2020
1 parent 769335e commit 7a071a9
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 1 deletion.
2 changes: 2 additions & 0 deletions include/sysemu/sysemu.h
Expand Up @@ -15,6 +15,8 @@ extern const char *qemu_name;
extern QemuUUID qemu_uuid;
extern bool qemu_uuid_set;

void qemu_add_data_dir(const char *path);

void qemu_add_exit_notifier(Notifier *notify);
void qemu_remove_exit_notifier(Notifier *notify);

Expand Down
2 changes: 1 addition & 1 deletion softmmu/vl.c
Expand Up @@ -1993,7 +1993,7 @@ char *qemu_find_file(int type, const char *name)
return NULL;
}

static void qemu_add_data_dir(const char *path)
void qemu_add_data_dir(const char *path)
{
int i;

Expand Down
15 changes: 15 additions & 0 deletions tests/qtest/fuzz/fuzz.c
Expand Up @@ -137,6 +137,7 @@ int LLVMFuzzerInitialize(int *argc, char ***argv, char ***envp)
{

char *target_name;
char *dir;

/* Initialize qgraph and modules */
qos_graph_init();
Expand All @@ -147,6 +148,20 @@ int LLVMFuzzerInitialize(int *argc, char ***argv, char ***envp)
target_name = strstr(**argv, "-target-");
if (target_name) { /* The binary name specifies the target */
target_name += strlen("-target-");
/*
* With oss-fuzz, the executable is kept in the root of a directory (we
* cannot assume the path). All data (including bios binaries) must be
* in the same dir, or a subdir. Thus, we cannot place the pc-bios so
* that it would be in exec_dir/../pc-bios.
* As a workaround, oss-fuzz allows us to use argv[0] to get the
* location of the executable. Using this we add exec_dir/pc-bios to
* the datadirs.
*/
dir = g_build_filename(g_path_get_dirname(**argv), "pc-bios", NULL);
if (g_file_test(dir, G_FILE_TEST_IS_DIR)) {
qemu_add_data_dir(dir);
}
g_free(dir);
} else if (*argc > 1) { /* The target is specified as an argument */
target_name = (*argv)[1];
if (!strstr(target_name, "--fuzz-target=")) {
Expand Down

0 comments on commit 7a071a9

Please sign in to comment.