- command=strprintf("convert -depth 8 -size %dx%d rgb%s:-[0] -density %dx%d \"%s\"\n",
- desc.get_w(), desc.get_h(), // size
- ((channels(pf) == 4) ? "a" : ""), // rgba or rgb?
- round_to_int(desc.get_x_res()/39.3700787402), // density
- round_to_int(desc.get_y_res()/39.3700787402),
- newfilename.c_str());
-
- file=popen(command.c_str(),POPEN_BINARY_WRITE_TYPE);
+ int p[2];
+
+ if (pipe(p)) {
+ if(cb) cb->error(N_(msg));
+ else synfig::error(N_(msg));
+ return false;
+ };
+
+ pid = fork();
+
+ if (pid == -1) {
+ if(cb) cb->error(N_(msg));
+ else synfig::error(N_(msg));
+ return false;
+ }
+
+ if (pid == 0){
+ // Child process
+ // Close pipeout, not needed
+ close(p[1]);
+ // Dup pipeout to stdin
+ if( dup2( p[0], STDIN_FILENO ) == -1 ){
+ if(cb) cb->error(N_(msg));
+ else synfig::error(N_(msg));
+ return false;
+ }
+ // Close the unneeded pipeout
+ close(p[0]);
+ execlp("convert", "convert",
+ "-depth", "8",
+ "-size", strprintf("%dx%d", desc.get_w(), desc.get_h()).c_str(),
+ ((channels(pf) == 4) ? "rgba:-[0]" : "rgb:-[0]"),
+ "-density", strprintf("%dx%d", round_to_int(desc.get_x_res()/39.3700787402), round_to_int(desc.get_y_res()/39.3700787402)).c_str(),
+ newfilename.c_str(),
+ (const char *)NULL);
+ // We should never reach here unless the exec failed
+ if(cb) cb->error(N_(msg));
+ else synfig::error(N_(msg));
+ return false;
+ } else {
+ // Parent process
+ // Close pipein, not needed
+ close(p[0]);
+ // Save pipeout to file handle, will write to it later
+ file = fdopen(p[1], "wb");
+ }