StarMirror/sandboxed-api
1
0
Fork 0
mirror of https://github.com/google/sandboxed-api.git synced 2024-03-22 13:11:30 +08:00
Code Issues Projects Releases Wiki Activity

added relative path cleanup using file::CleanPath

Browse Source
This commit is contained in:
Andrei Medar 2020-09-24 18:47:19 +00:00
parent 388bcff972
commit 3052967374
3 changed files with 18 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
oss-internship-2020/sapi_libarchive/examples/helpers.cc
View File

@ -15,11 +15,11 @@ std::vector<std::string> MakeAbsolutePathsVec(const char* argv[]) {
return arr; return arr;
} }
std::string CheckStatusAndGetString(const sapi::StatusOr<char*>& status, std::string CheckStatusAndGetString(const absl::StatusOr<char*>& status,
LibarchiveSandbox& sandbox) { LibarchiveSandbox& sandbox) {
CHECK(status.ok() && status.value() != NULL) << "Could not get error message"; CHECK(status.ok() && status.value() != NULL) << "Could not get error message";
sapi::StatusOr<std::string> ret = absl::StatusOr<std::string> ret =
sandbox.GetCString(sapi::v::RemotePtr(status.value())); sandbox.GetCString(sapi::v::RemotePtr(status.value()));
CHECK(ret.ok()) << "Could not transfer error message"; CHECK(ret.ok()) << "Could not transfer error message";
return ret.value(); return ret.value();
@ -30,7 +30,7 @@ std::string CreateTempDirAtCWD() {
CHECK(!cwd.empty()) << "Could not get current working directory"; CHECK(!cwd.empty()) << "Could not get current working directory";
cwd.append("/"); cwd.append("/");
sapi::StatusOr<std::string> result = sandbox2::CreateTempDir(cwd); absl::StatusOr<std::string> result = sandbox2::CreateTempDir(cwd);
CHECK(result.ok()) << "Could not create temporary directory at " << cwd; CHECK(result.ok()) << "Could not create temporary directory at " << cwd;
return result.value(); return result.value();
} }

2
oss-internship-2020/sapi_libarchive/examples/helpers.h
View File

@ -24,7 +24,7 @@ std::string MakeAbsolutePathAtCWD(const std::string& path);
// This function takes a status as argument and after checking the status // This function takes a status as argument and after checking the status
// it transfers the string. This is used mostly with archive_error_string // it transfers the string. This is used mostly with archive_error_string
// and other library functions that return a char *. // and other library functions that return a char *.
std::string CheckStatusAndGetString(const sapi::StatusOr<char*>& status, std::string CheckStatusAndGetString(const absl::StatusOr<char*>& status,
LibarchiveSandbox& sandbox); LibarchiveSandbox& sandbox);
// Creates a temporary directory in the current working directory and // Creates a temporary directory in the current working directory and

24
oss-internship-2020/sapi_libarchive/examples/sapi_minitar.cc
View File

@ -240,17 +240,20 @@ static void create(const char* initial_filename, int compress,
sandbox2::util::CharPtrArrToVecString(const_cast<char* const*>(argv), sandbox2::util::CharPtrArrToVecString(const_cast<char* const*>(argv),
&relative_paths); &relative_paths);
std::transform(relative_paths.begin(), relative_paths.end(),
relative_paths.begin(), sandbox2::file::CleanPath);
SapiLibarchiveSandboxCreate sandbox(absolute_paths, archive_path); SapiLibarchiveSandboxCreate sandbox(absolute_paths, archive_path);
CHECK(sandbox.Init().ok()) << "Error during sandbox initialization"; CHECK(sandbox.Init().ok()) << "Error during sandbox initialization";
LibarchiveApi api(&sandbox); LibarchiveApi api(&sandbox);
sapi::StatusOr<archive*> ret = api.archive_write_new(); absl::StatusOr<archive*> ret = api.archive_write_new();
CHECK(ret.ok()) << "write_new call failed"; CHECK(ret.ok()) << "write_new call failed";
CHECK(ret.value() != NULL) << "Failed to create write archive"; CHECK(ret.value() != NULL) << "Failed to create write archive";
sapi::v::RemotePtr a(ret.value()); sapi::v::RemotePtr a(ret.value());
sapi::StatusOr<int> ret2; absl::StatusOr<int> ret2;
switch (compress) { switch (compress) {
#ifndef NO_BZIP2_CREATE #ifndef NO_BZIP2_CREATE
@ -328,7 +331,7 @@ static void create(const char* initial_filename, int compress,
for (;;) { for (;;) {
int needcr = 0; int needcr = 0;
sapi::StatusOr<archive_entry*> ret3; absl::StatusOr<archive_entry*> ret3;
ret3 = api.archive_entry_new(); ret3 = api.archive_entry_new();
CHECK(ret3.ok()) << "entry_new call failed"; CHECK(ret3.ok()) << "entry_new call failed";
@ -401,11 +404,10 @@ static void create(const char* initial_filename, int compress,
<< "Unexpected result from write_header call"; << "Unexpected result from write_header call";
if (ret2.value() > ARCHIVE_FAILED) { if (ret2.value() > ARCHIVE_FAILED) {
// int fd = open(CheckStatusAndGetString( int fd = open(CheckStatusAndGetString(
// api.archive_entry_sourcepath(&entry), sandbox) api.archive_entry_sourcepath(&entry), sandbox)
// .c_str(), .c_str(),
// O_RDONLY); O_RDONLY);
int fd = open(path_name.c_str(), O_RDONLY);
CHECK(fd >= 0) << "Could not open file"; CHECK(fd >= 0) << "Could not open file";
sapi::v::Fd sapi_fd(fd); sapi::v::Fd sapi_fd(fd);
@ -493,7 +495,7 @@ static void extract(const char* filename, int do_extract, int flags) {
LibarchiveApi api(&sandbox); LibarchiveApi api(&sandbox);
sapi::StatusOr<archive*> ret = api.archive_read_new(); absl::StatusOr<archive*> ret = api.archive_read_new();
CHECK(ret.ok()) << "archive_read_new call failed"; CHECK(ret.ok()) << "archive_read_new call failed";
CHECK(ret.value() != NULL) << "Failed to create read archive"; CHECK(ret.value() != NULL) << "Failed to create read archive";
@ -505,7 +507,7 @@ static void extract(const char* filename, int do_extract, int flags) {
sapi::v::RemotePtr ext(ret.value()); sapi::v::RemotePtr ext(ret.value());
sapi::StatusOr<int> ret2; absl::StatusOr<int> ret2;
ret2 = api.archive_write_disk_set_options(&ext, flags); ret2 = api.archive_write_disk_set_options(&ext, flags);
CHECK(ret2.ok()) << "write_disk_set_options call failed"; CHECK(ret2.ok()) << "write_disk_set_options call failed";
CHECK(ret2.value() != ARCHIVE_FATAL) CHECK(ret2.value() != ARCHIVE_FATAL)
@ -624,7 +626,7 @@ static void extract(const char* filename, int do_extract, int flags) {
static int copy_data(sapi::v::RemotePtr* ar, sapi::v::RemotePtr* aw, static int copy_data(sapi::v::RemotePtr* ar, sapi::v::RemotePtr* aw,
LibarchiveApi& api, LibarchiveApi& api,
SapiLibarchiveSandboxExtract& sandbox) { SapiLibarchiveSandboxExtract& sandbox) {
sapi::StatusOr<int> ret; absl::StatusOr<int> ret;
sapi::v::IntBase<struct archive_entry*> buff_ptr_tmp(0); sapi::v::IntBase<struct archive_entry*> buff_ptr_tmp(0);
sapi::v::ULLong size; sapi::v::ULLong size;