Do not run pre*export* functions in the pre*build* hook

[apt.git] / cmdline / apt-dump-solver.cc

diff --git a/cmdline/apt-dump-solver.cc b/cmdline/apt-dump-solver.cc
index f765234c5f8b61d5fc2a729e76d39699b277e047..27592f22eb5f3e5258785f27cd9fd3300735f53d 100644 (file)
--- a/cmdline/apt-dump-solver.cc
+++ b/cmdline/apt-dump-solver.cc
@@ -13,6 +13,7 @@
 #include <unistd.h>
 #include <cstdio>
 #include <iostream>
+#include <sstream>
 
 #include <config.h>
                                                                        /*}}}*/
@@ -21,13 +22,13 @@
 // ---------------------------------------------------------------------
 /* */
 static bool ShowHelp() {
-
+       ioprintf(std::cout, "%s %s (%s)\n", PACKAGE, PACKAGE_VERSION, COMMON_ARCH);
        std::cout <<
-               PACKAGE " " PACKAGE_VERSION " for " COMMON_ARCH " compiled on " __DATE__ " " __TIME__ << std::endl <<
-               "Usage: apt-dump-resolver\n"
+               "Usage: apt-dump-solver\n"
                "\n"
-               "apt-dump-resolver is a dummy solver who just dumps its input to the\n"
-               "file /tmp/dump.edsp and exists with a proper EDSP error.\n"
+               "apt-dump-solver is a dummy solver who just dumps its input to the\n"
+               "file specified in the environment variable APT_EDSP_DUMP_FILENAME and\n"
+               "exists with a proper EDSP error.\n"
                "\n"
                "                       This dump has lost Super Cow Powers.\n";
        return true;
@@ -40,16 +41,31 @@ int main(int argc,const char *argv[])                                       /*{{{*/
                ShowHelp();
                return 0;
        }
-        // we really don't need anything
-        DropPrivileges();
 
-       FILE* input = fdopen(STDIN_FILENO, "r");
-       FILE* output = fopen("/tmp/dump.edsp", "w");
-       char buffer[400];
-       while (fgets(buffer, sizeof(buffer), input) != NULL)
-               fputs(buffer, output);
-       fclose(output);
-       fclose(input);
+       // we really don't need anything
+       DropPrivileges();
+       char const * const filename = getenv("APT_EDSP_DUMP_FILENAME");
+       if (filename == NULL || strlen(filename) == 0)
+       {
+          EDSP::WriteError("ERR_NO_FILENAME", "You have to set the environment variable APT_EDSP_DUMP_FILENAME\n"
+                "to a valid filename to store the dump of EDSP solver input in.\n"
+                "For example with: export APT_EDSP_DUMP_FILENAME=/tmp/dump.edsp", stdout);
+          return 0;
+       }
+
+       RemoveFile(argv[0], filename);
+       FileFd input, output;
+       if (input.OpenDescriptor(STDIN_FILENO, FileFd::ReadOnly) == false ||
+             output.Open(filename, FileFd::WriteOnly | FileFd::Create | FileFd::Exclusive, 0600) == false ||
+             CopyFile(input, output) == false || input.Close() == false || output.Close() == false)
+       {
+          std::ostringstream out;
+          out << "Writing EDSP solver input to file '" << filename << "' failed!\n";
+          _error->DumpErrors(out);
+          EDSP::WriteError("ERR_WRITE_ERROR", out.str(), stdout);
+          return 0;
+       }
 
        EDSP::WriteError("ERR_JUST_DUMPING", "I am too dumb, i can just dump!\nPlease use one of my friends instead!", stdout);
+       return 0;
 }