- use bb_xchdir instead of ignoring eventual errors of chdir

  Thanks to Erik Hovland

diff --git a/archival/dpkg_deb.c b/archival/dpkg_deb.c
index b99f227fcfa8f7f111e68fbd1f7bfb9dee6c6e05..48a1ac16135e7f6d9b52cf0f2ece9a4507fd79d7 100644 (file)
--- a/archival/dpkg_deb.c
+++ b/archival/dpkg_deb.c
@@ -89,8 +89,8 @@ int dpkg_deb_main(int argc, char **argv)
                extract_dir = argv[optind];
        }
        if (extract_dir) {
-               mkdir(extract_dir, 0777);
-               chdir(extract_dir); /* error check? */
+               mkdir(extract_dir, 0777); /* bb_make_directory(extract_dir, 0777, 0) */
+               bb_xchdir(extract_dir);
        }
        unpack_ar_archive(ar_archive);
 

diff --git a/archival/rpm.c b/archival/rpm.c
index 9bbc4a8567ccc092cc49a67d9dbc99b0d2cbb77e..0755789714f3953c6d36c91e0e1c696c69c829ff 100644 (file)
--- a/archival/rpm.c
+++ b/archival/rpm.c
@@ -198,7 +198,7 @@ void extract_cpio_gz(int fd) {
                bb_error_msg_and_die("Invalid gzip magic");
        }
        check_header_gzip(archive_handle->src_fd);
-       chdir("/"); // Install RPM's to root
+       bb_xchdir("/"); // Install RPM's to root
 
        archive_handle->src_fd = open_transformer(archive_handle->src_fd, inflate_gunzip);
        archive_handle->offset = 0;