X-Git-Url: https://wimlib.net/git/?a=blobdiff_plain;f=src%2Fextract.c;h=aba92865ba9c3f2bee00175b4a8ec2cf8eff26d6;hb=d9675dd5814394373d9871c6e9b7b35325b3d21d;hp=8f658c4f7cd96cd10d1f6defc5e848069a546af3;hpb=3ffb2cde078ae8f62d542ab89166e1059c13d758;p=wimlib diff --git a/src/extract.c b/src/extract.c index 8f658c4f..aba92865 100644 --- a/src/extract.c +++ b/src/extract.c @@ -49,6 +49,7 @@ #include "wimlib/error.h" #include "wimlib/lookup_table.h" #include "wimlib/metadata.h" +#include "wimlib/pathlist.h" #include "wimlib/paths.h" #include "wimlib/reparse.h" #include "wimlib/resource.h" @@ -57,6 +58,7 @@ # include "wimlib/win32.h" /* for realpath() equivalent */ #endif #include "wimlib/xml.h" +#include "wimlib/wildcard.h" #include "wimlib/wim.h" #include @@ -140,7 +142,7 @@ ref_stream_to_extract(struct wim_lookup_table_entry *lte, ctx->num_streams_remaining++; } - if (ctx->extract_flags & WIMLIB_EXTRACT_FLAG_SEQUENTIAL) { + if (!(ctx->extract_flags & WIMLIB_EXTRACT_FLAG_FILE_ORDER)) { struct wim_dentry **lte_dentries; /* Append dentry to this stream's array of dentries referencing @@ -165,7 +167,7 @@ ref_stream_to_extract(struct wim_lookup_table_entry *lte, lte_dentries = REALLOC(prev_lte_dentries, alloc_lte_dentries * sizeof(lte_dentries[0])); - if (!lte_dentries) + if (lte_dentries == NULL) return WIMLIB_ERR_NOMEM; if (prev_lte_dentries == NULL) { memcpy(lte_dentries, @@ -534,10 +536,10 @@ error: * This function operates slightly differently depending on whether @lte_spec is * NULL or not. When @lte_spec is NULL, the behavior is to extract the default * file contents (unnamed stream), and, if named data streams are supported in - * the extract mode and volume, any named data streams. When @lte_spec is NULL, - * the behavior is to extract only all copies of the stream @lte_spec, and in - * addition use @lte_spec to set the reparse data or create the symbolic link if - * appropriate. + * the extract mode and volume, any named data streams. When @lte_spec is not + * NULL, the behavior is to extract only all copies of the stream @lte_spec, and + * in addition use @lte_spec to set the reparse data or create the symbolic link + * if appropriate. * * @path * Path to file to extract (as can be passed to apply_operations @@ -1182,8 +1184,7 @@ dentry_extract_dir_skeleton(struct wim_dentry *dentry, void *_ctx) return 0; } -/* Create a file or directory, then immediately extract all streams. This - * assumes that WIMLIB_EXTRACT_FLAG_SEQUENTIAL is not specified, since the WIM +/* Create a file or directory, then immediately extract all streams. The WIM * may not be read sequentially by this function. */ static int dentry_extract(struct wim_dentry *dentry, void *_ctx) @@ -1202,23 +1203,50 @@ dentry_extract(struct wim_dentry *dentry, void *_ctx) return extract_streams(path, ctx, dentry, NULL, NULL); } +/* Creates a temporary file opened for writing. The open file descriptor is + * returned in @fd_ret and its name is returned in @name_ret (dynamically + * allocated). */ +static int +create_temporary_file(struct filedes *fd_ret, tchar **name_ret) +{ + tchar *name; + int raw_fd; + +retry: + name = ttempnam(NULL, T("wimlib")); + if (name == NULL) { + ERROR_WITH_ERRNO("Failed to create temporary filename"); + return WIMLIB_ERR_NOMEM; + } + + raw_fd = topen(name, O_WRONLY | O_CREAT | O_EXCL | O_BINARY, 0600); + + if (raw_fd < 0) { + if (errno == EEXIST) { + FREE(name); + goto retry; + } + ERROR_WITH_ERRNO("Failed to open temporary file \"%"TS"\"", name); + FREE(name); + return WIMLIB_ERR_OPEN; + } + + filedes_init(fd_ret, raw_fd); + *name_ret = name; + return 0; +} + /* Extract all instances of the stream @lte that are being extracted in this - * call of extract_tree(). @can_seek specifies whether the WIM file descriptor - * is seekable or not (e.g. is a pipe). If not and the stream needs to be - * extracted multiple times, it is extracted to a temporary file first. - * - * This is intended for use with sequential extraction of a WIM image - * (WIMLIB_EXTRACT_FLAG_SEQUENTIAL specified). */ + * call of extract_tree(), but actually read the stream data from @lte_override. + */ static int extract_stream_instances(struct wim_lookup_table_entry *lte, - struct apply_ctx *ctx, bool can_seek) + struct wim_lookup_table_entry *lte_override, + struct apply_ctx *ctx) { struct wim_dentry **lte_dentries; - struct wim_lookup_table_entry *lte_tmp = NULL; - struct wim_lookup_table_entry *lte_override; - tchar *stream_tmp_filename = NULL; tchar path[ctx->ops->path_max]; - unsigned i; + size_t i; int ret; if (lte->out_refcnt <= ARRAY_LEN(lte->inline_lte_dentries)) @@ -1226,49 +1254,6 @@ extract_stream_instances(struct wim_lookup_table_entry *lte, else lte_dentries = lte->lte_dentries; - if (likely(can_seek || lte->out_refcnt < 2)) { - lte_override = lte; - } else { - /* Need to extract stream to temporary file. */ - struct filedes fd; - int raw_fd; - - stream_tmp_filename = ttempnam(NULL, T("wimlib")); - if (!stream_tmp_filename) { - ERROR_WITH_ERRNO("Failed to create temporary filename"); - ret = WIMLIB_ERR_OPEN; - goto out; - } - - lte_tmp = memdup(lte, sizeof(struct wim_lookup_table_entry)); - if (!lte_tmp) { - ret = WIMLIB_ERR_NOMEM; - goto out_free_stream_tmp_filename; - } - lte_tmp->resource_location = RESOURCE_IN_FILE_ON_DISK; - lte_tmp->file_on_disk = stream_tmp_filename; - lte_override = lte_tmp; - - raw_fd = topen(stream_tmp_filename, - O_WRONLY | O_CREAT | O_TRUNC | O_BINARY, 0600); - if (raw_fd < 0) { - ERROR_WITH_ERRNO("Failed to open temporary file"); - ret = WIMLIB_ERR_OPEN; - goto out_free_lte_tmp; - } - filedes_init(&fd, raw_fd); - ret = extract_stream_to_fd(lte, &fd, lte->size); - if (filedes_close(&fd) && !ret) - ret = WIMLIB_ERR_WRITE; - if (ret) - goto out_unlink_stream_tmp_file; - } - - /* Extract all instances of the stream, reading either from the stream - * in the WIM file or from the temporary file containing the stream. - * dentry->tmp_flag is used to ensure that each dentry is processed only - * once regardless of how many times this stream appears in the streams - * of the corresponding inode. */ for (i = 0; i < lte->out_refcnt; i++) { struct wim_dentry *dentry = lte_dentries[i]; @@ -1276,8 +1261,7 @@ extract_stream_instances(struct wim_lookup_table_entry *lte, continue; if (!build_extraction_path(path, dentry, ctx)) continue; - ret = extract_streams(path, ctx, dentry, - lte, lte_override); + ret = extract_streams(path, ctx, dentry, lte, lte_override); if (ret) goto out_clear_tmp_flags; dentry->tmp_flag = 1; @@ -1286,38 +1270,138 @@ extract_stream_instances(struct wim_lookup_table_entry *lte, out_clear_tmp_flags: for (i = 0; i < lte->out_refcnt; i++) lte_dentries[i]->tmp_flag = 0; -out_unlink_stream_tmp_file: - if (stream_tmp_filename) - tunlink(stream_tmp_filename); -out_free_lte_tmp: - FREE(lte_tmp); -out_free_stream_tmp_filename: - FREE(stream_tmp_filename); -out: + return ret; +} + +/* Determine whether the specified stream needs to be extracted to a temporary + * file or not. + * + * @lte->out_refcnt specifies the number of instances of this stream that must + * be extracted. + * + * @is_partial_res is %true if this stream is just one of multiple in a single + * WIM resource being extracted. */ +static bool +need_tmpfile_to_extract(struct wim_lookup_table_entry *lte, + bool is_partial_res) +{ + /* Temporary file is always required when reading a partial resource, + * since in that case we retrieve all the contained streams in one pass. + * */ + if (is_partial_res) + return true; + + /* Otherwise we don't need a temporary file if only a single instance of + * the stream is needed. */ + if (lte->out_refcnt == 1) + return false; + + wimlib_assert(lte->out_refcnt >= 2); + + /* We also don't need a temporary file if random access to the stream is + * allowed. */ + if (lte->resource_location != RESOURCE_IN_WIM || + filedes_is_seekable(<e->rspec->wim->in_fd)) + return false; + + return true; +} + +static int +begin_extract_stream_to_tmpfile(struct wim_lookup_table_entry *lte, + bool is_partial_res, + void *_ctx) +{ + struct apply_ctx *ctx = _ctx; + int ret; + + if (!need_tmpfile_to_extract(lte, is_partial_res)) { + DEBUG("Temporary file not needed " + "for stream (size=%"PRIu64")", lte->size); + ret = extract_stream_instances(lte, lte, ctx); + if (ret) + return ret; + + return BEGIN_STREAM_STATUS_SKIP_STREAM; + } + + DEBUG("Temporary file needed for stream (size=%"PRIu64")", lte->size); + return create_temporary_file(&ctx->tmpfile_fd, &ctx->tmpfile_name); +} + +static int +end_extract_stream_to_tmpfile(struct wim_lookup_table_entry *lte, + int status, void *_ctx) +{ + struct apply_ctx *ctx = _ctx; + struct wim_lookup_table_entry lte_override; + int ret; + int errno_save = errno; + + ret = filedes_close(&ctx->tmpfile_fd); + + if (status) { + ret = status; + errno = errno_save; + goto out_delete_tmpfile; + } + + if (ret) { + ERROR_WITH_ERRNO("Error writing temporary file %"TS, ctx->tmpfile_name); + ret = WIMLIB_ERR_WRITE; + goto out_delete_tmpfile; + } + + /* Now that a full stream has been extracted to a temporary file, + * extract all instances of it to the actual target. */ + + memcpy(<e_override, lte, sizeof(struct wim_lookup_table_entry)); + lte_override.resource_location = RESOURCE_IN_FILE_ON_DISK; + lte_override.file_on_disk = ctx->tmpfile_name; + + ret = extract_stream_instances(lte, <e_override, ctx); + +out_delete_tmpfile: + errno_save = errno; + tunlink(ctx->tmpfile_name); + FREE(ctx->tmpfile_name); + errno = errno_save; return ret; } /* Extracts a list of streams (ctx.stream_list), assuming that the directory * structure and empty files were already created. This relies on the * per-`struct wim_lookup_table_entry' list of dentries that reference each - * stream that was constructed earlier. Streams are extracted exactly in the - * order of the stream list; however, unless the WIM's file descriptor is - * detected to be non-seekable, streams may be read from the WIM file more than - * one time if multiple copies need to be extracted. */ + * stream that was constructed earlier. */ static int extract_stream_list(struct apply_ctx *ctx) { - struct wim_lookup_table_entry *lte; - bool can_seek; - int ret; + if (!(ctx->extract_flags & WIMLIB_EXTRACT_FLAG_FILE_ORDER)) { + /* Sequential extraction: read the streams in the order in which + * they appear in the WIM file. */ + struct read_stream_list_callbacks cbs = { + .begin_stream = begin_extract_stream_to_tmpfile, + .begin_stream_ctx = ctx, + .consume_chunk = extract_chunk_to_fd, + .consume_chunk_ctx = &ctx->tmpfile_fd, + .end_stream = end_extract_stream_to_tmpfile, + .end_stream_ctx = ctx, + }; + return read_stream_list(&ctx->stream_list, + offsetof(struct wim_lookup_table_entry, extraction_list), + &cbs, VERIFY_STREAM_HASHES); + } else { + /* Extract the streams in unsorted order. */ + struct wim_lookup_table_entry *lte; + int ret; - can_seek = (lseek(ctx->wim->in_fd.fd, 0, SEEK_CUR) != -1); - list_for_each_entry(lte, &ctx->stream_list, extraction_list) { - ret = extract_stream_instances(lte, ctx, can_seek); - if (ret) - return ret; + list_for_each_entry(lte, &ctx->stream_list, extraction_list) { + ret = extract_stream_instances(lte, lte, ctx); + if (ret) + return ret; + } + return 0; } - return 0; } #define PWM_ALLOW_WIM_HDR 0x00001 @@ -1352,7 +1436,7 @@ read_pwm_stream_header(WIMStruct *pwm, struct wim_lookup_table_entry *lte, return 0; } - if (buf.stream_hdr.magic != PWM_STREAM_MAGIC) { + if (le64_to_cpu(buf.stream_hdr.magic) != PWM_STREAM_MAGIC) { ERROR("Data read on pipe is invalid (expected stream header)."); return WIMLIB_ERR_INVALID_PIPABLE_WIM; } @@ -1376,25 +1460,6 @@ read_error: return ret; } -static int -skip_pwm_chunk_cb(const void *chunk, size_t chunk_size, void *_ctx) -{ - return 0; -} - -/* Skip over an unneeded stream in a pipable WIM being read from a pipe. */ -static int -skip_pwm_stream(struct wim_lookup_table_entry *lte) -{ - return read_partial_wim_resource(lte, - lte->size, - skip_pwm_chunk_cb, - lte_cchunk_size(lte), - NULL, - WIMLIB_READ_RESOURCE_FLAG_RAW_CHUNKS, - 0); -} - static int extract_streams_from_pipe(struct apply_ctx *ctx) { @@ -1444,16 +1509,60 @@ extract_streams_from_pipe(struct apply_ctx *ctx) && (needed_lte = lookup_resource(lookup_table, found_lte->hash)) && (needed_lte->out_refcnt)) { + tchar *tmpfile_name = NULL; + struct wim_lookup_table_entry *lte_override; + struct wim_lookup_table_entry tmpfile_lte; + + needed_lte->offset_in_res = found_lte->offset_in_res; + needed_lte->flags = found_lte->flags; + needed_lte->size = found_lte->size; + lte_unbind_wim_resource_spec(found_lte); lte_bind_wim_resource_spec(needed_lte, rspec); - ret = extract_stream_instances(needed_lte, ctx, false); - lte_unbind_wim_resource_spec(needed_lte); + if (needed_lte->out_refcnt > 1) { + + struct filedes tmpfile_fd; + + /* Extract stream to temporary file. */ + ret = create_temporary_file(&tmpfile_fd, &tmpfile_name); + if (ret) + goto out_free_found_lte; + + ret = extract_stream_to_fd(needed_lte, &tmpfile_fd, + needed_lte->size); + if (ret) { + filedes_close(&tmpfile_fd); + goto delete_tmpfile; + } + + if (filedes_close(&tmpfile_fd)) { + ERROR_WITH_ERRNO("Error writing to temporary " + "file \"%"TS"\"", tmpfile_name); + ret = WIMLIB_ERR_WRITE; + goto delete_tmpfile; + } + memcpy(&tmpfile_lte, needed_lte, + sizeof(struct wim_lookup_table_entry)); + tmpfile_lte.resource_location = RESOURCE_IN_FILE_ON_DISK; + tmpfile_lte.file_on_disk = tmpfile_name; + lte_override = &tmpfile_lte; + } else { + lte_override = needed_lte; + } + + ret = extract_stream_instances(needed_lte, lte_override, ctx); + delete_tmpfile: + lte_unbind_wim_resource_spec(needed_lte); + if (tmpfile_name) { + tunlink(tmpfile_name); + FREE(tmpfile_name); + } if (ret) goto out_free_found_lte; ctx->num_streams_remaining--; } else if (found_lte->resource_location != RESOURCE_NONEXISTENT) { - ret = skip_pwm_stream(found_lte); + ret = skip_wim_stream(found_lte); if (ret) goto out_free_found_lte; } else { @@ -2265,21 +2374,6 @@ extract_tree(WIMStruct *wim, const tchar *wim_source_path, const tchar *target, goto out_teardown_stream_list; } - /* If a sequential extraction was specified, sort the streams to be - * extracted by their position in the WIM file so that the WIM file can - * be read sequentially. */ - if ((extract_flags & (WIMLIB_EXTRACT_FLAG_SEQUENTIAL | - WIMLIB_EXTRACT_FLAG_FROM_PIPE)) - == WIMLIB_EXTRACT_FLAG_SEQUENTIAL) - { - ret = sort_stream_list_by_sequential_order( - &ctx.stream_list, - offsetof(struct wim_lookup_table_entry, - extraction_list)); - if (ret) - goto out_teardown_stream_list; - } - if (ctx.ops->realpath_works_on_nonexisting_files && ((extract_flags & WIMLIB_EXTRACT_FLAG_RPFIX) || ctx.ops->requires_realtarget_in_paths)) @@ -2337,8 +2431,7 @@ extract_tree(WIMStruct *wim, const tchar *wim_source_path, const tchar *target, } /* Finally, the important part: extract the tree of files. */ - if (extract_flags & (WIMLIB_EXTRACT_FLAG_SEQUENTIAL | - WIMLIB_EXTRACT_FLAG_FROM_PIPE)) { + if (!(extract_flags & WIMLIB_EXTRACT_FLAG_FILE_ORDER)) { /* Sequential extraction requested, so two passes are needed * (one for directory structure, one for streams.) */ if (progress_func) @@ -2416,7 +2509,7 @@ out_free_realtarget: out_teardown_stream_list: /* Free memory allocated as part of the mapping from each * wim_lookup_table_entry to the dentries that reference it. */ - if (ctx.extract_flags & WIMLIB_EXTRACT_FLAG_SEQUENTIAL) + if (!(ctx.extract_flags & WIMLIB_EXTRACT_FLAG_FILE_ORDER)) list_for_each_entry(lte, &ctx.stream_list, extraction_list) if (lte->out_refcnt > ARRAY_LEN(lte->inline_lte_dentries)) FREE(lte->lte_dentries); @@ -2454,6 +2547,9 @@ check_extract_command(struct wimlib_extract_command *cmd, int wim_header_flags) WIMLIB_EXTRACT_FLAG_HARDLINK)) return WIMLIB_ERR_INVALID_PARAM; + if (extract_flags & WIMLIB_EXTRACT_FLAG_GLOB_PATHS) + return WIMLIB_ERR_INVALID_PARAM; + if ((extract_flags & (WIMLIB_EXTRACT_FLAG_NO_ACLS | WIMLIB_EXTRACT_FLAG_STRICT_ACLS)) == (WIMLIB_EXTRACT_FLAG_NO_ACLS | @@ -2493,9 +2589,8 @@ check_extract_command(struct wimlib_extract_command *cmd, int wim_header_flags) * corresponding file or symbolic link data. This needs to be handled * better. */ if ((extract_flags & (WIMLIB_EXTRACT_FLAG_UNIX_DATA | - WIMLIB_EXTRACT_FLAG_SEQUENTIAL)) - == (WIMLIB_EXTRACT_FLAG_UNIX_DATA | - WIMLIB_EXTRACT_FLAG_SEQUENTIAL)) + WIMLIB_EXTRACT_FLAG_FILE_ORDER)) + == WIMLIB_EXTRACT_FLAG_UNIX_DATA) { if (extract_flags & WIMLIB_EXTRACT_FLAG_FROM_PIPE) { WARNING("Setting UNIX file/owner group may " @@ -2503,7 +2598,7 @@ check_extract_command(struct wimlib_extract_command *cmd, int wim_header_flags) " symbolic links " "when applying from a pipe."); } else { - extract_flags &= ~WIMLIB_EXTRACT_FLAG_SEQUENTIAL; + extract_flags |= WIMLIB_EXTRACT_FLAG_FILE_ORDER; WARNING("Disabling sequential extraction for " "UNIX data mode"); } @@ -2785,8 +2880,6 @@ wimlib_extract_image_from_pipe(int pipe_fd, const tchar *image_num_or_name, if (extract_flags & WIMLIB_EXTRACT_FLAG_TO_STDOUT) return WIMLIB_ERR_INVALID_PARAM; - extract_flags |= WIMLIB_EXTRACT_FLAG_SEQUENTIAL; - /* Read the WIM header from the pipe and get a WIMStruct to represent * the pipable WIM. Caveats: Unlike getting a WIMStruct with * wimlib_open_wim(), getting a WIMStruct in this way will result in @@ -2917,7 +3010,7 @@ wimlib_extract_image_from_pipe(int pipe_fd, const tchar *image_num_or_name, } else { /* Metadata resource is not for the image being * extracted. Skip over it. */ - ret = skip_pwm_stream(metadata_lte); + ret = skip_wim_stream(metadata_lte); if (ret) goto out_wimlib_free; } @@ -2944,3 +3037,122 @@ wimlib_extract_image(WIMStruct *wim, return do_wimlib_extract_image(wim, image, target, extract_flags, progress_func); } + +/* API function documented in wimlib.h */ +WIMLIBAPI int +wimlib_extract_pathlist(WIMStruct *wim, int image, + const tchar *target, + const tchar *path_list_file, + int extract_flags, + wimlib_progress_func_t progress_func) +{ + int ret; + tchar **paths; + size_t num_paths; + void *mem; + + ret = read_path_list_file(path_list_file, &paths, &num_paths, &mem); + if (ret) + return ret; + + ret = wimlib_extract_paths(wim, image, target, + (const tchar * const *)paths, num_paths, + extract_flags, progress_func); + FREE(paths); + FREE(mem); + return ret; +} + +/* API function documented in wimlib.h */ +WIMLIBAPI int +wimlib_extract_paths(WIMStruct *wim, + int image, + const tchar *target, + const tchar * const *paths, + size_t num_paths, + int extract_flags, + wimlib_progress_func_t progress_func) +{ + int ret; + tchar **expanded_paths; + size_t num_expanded_paths; + struct wimlib_extract_command *cmds; + + ret = select_wim_image(wim, image); + if (ret) + return ret; + + if (extract_flags & WIMLIB_EXTRACT_FLAG_GLOB_PATHS) { + int wildcard_flags = 0; + + if (extract_flags & WIMLIB_EXTRACT_FLAG_STRICT_GLOB) + wildcard_flags |= WILDCARD_FLAG_ERROR_IF_NO_MATCH; + else + wildcard_flags |= WILDCARD_FLAG_WARN_IF_NO_MATCH; + + if (extract_flags & WIMLIB_EXTRACT_FLAG_CASE_INSENSITIVE_GLOB) + wildcard_flags |= WILDCARD_FLAG_CASE_INSENSITIVE; + + ret = expand_wildcard_wim_paths(wim, paths, num_paths, + &expanded_paths, + &num_expanded_paths, + wildcard_flags); + if (ret) + return ret; + } else { + expanded_paths = (tchar**)paths; + num_expanded_paths = num_paths; + } + + cmds = CALLOC(num_expanded_paths, sizeof(cmds[0])); + if (cmds == NULL) { + ret = WIMLIB_ERR_NOMEM; + goto out_free_expanded_paths; + } + + for (size_t i = 0; i < num_expanded_paths; i++) { + cmds[i].wim_source_path = expanded_paths[i]; + cmds[i].extract_flags = 0; + + tchar *dest_path; + size_t dest_len = 0; + dest_len += tstrlen(target); + dest_len += 1; + dest_len += tstrlen(expanded_paths[i]); + dest_len += 1; + + dest_path = MALLOC(dest_len * sizeof(tchar)); + if (dest_path == NULL) { + ret = WIMLIB_ERR_NOMEM; + goto out_free_extraction_cmds; + } + tchar *p = dest_path; + p = tmempcpy(p, target, tstrlen(target)); + *p++ = OS_PREFERRED_PATH_SEPARATOR; + for (tchar *path_p = expanded_paths[i]; *path_p != '\0'; path_p++) { + if (is_any_path_separator(*path_p)) + *p++ = OS_PREFERRED_PATH_SEPARATOR; + else + *p++ = *path_p; + } + *p++ = T('\0'); + wimlib_assert(p - dest_path == dest_len); + cmds[i].fs_dest_path = dest_path; + } + + ret = wimlib_extract_files(wim, image, + cmds, num_expanded_paths, + extract_flags & ~WIMLIB_EXTRACT_FLAG_GLOB_PATHS, + progress_func); +out_free_extraction_cmds: + for (size_t i = 0; i < num_expanded_paths; i++) + FREE(cmds[i].fs_dest_path); + FREE(cmds); +out_free_expanded_paths: + if (extract_flags & WIMLIB_EXTRACT_FLAG_GLOB_PATHS) { + for (size_t i = 0; i < num_expanded_paths; i++) + FREE(expanded_paths[i]); + FREE(expanded_paths); + } + return ret; +}