Tue Feb 21 10:10:31 2012: Request 75193 was acted upon. Transaction: Correspondence added by RSCHUPP Queue: PAR-Packer Subject: [PATCH] Fix build on Win32/VC++ Broken in: 1.012 Severity: Important Owner: RSCHUPP Requestors: s...@cpan.org Status: open Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=75193 >
On 2012-02-21 09:32:44, SHAY wrote: > The current version (1.012) of PAR-Packer does not build on Win32 with > VC++. There are two problems: myldr/boot.c attempts to include unistd.h Thanks for the patch, these changes were already in trunk. > myldr\mktmpdir.c(45) : warning C4013: 'getuid' undefined; assuming > extern returning int I wrapped the whole function calling getuid (isSafeDir) in #ifndef WIN32 as it's only called on *nix anyway. OTOH if someone could implement isSafeDir for Windows, I surely would appreciate that :) It's there to fix a CVE "PAR::Packer creates per-user directory with unsafe permissions". I'm not sure what it takes to create a "safe per-user directory" on Windows, i.e. where only the user (and privileged accounts) can create and modify files. If you want to test the current snapshot (esp. since you use the VC++ toolchain), there were some recent changes in the resource handling for Win32 executables http://svn.openfoundry.org/par/PAR-Packer/trunk Cheers, Roderich