September 2008 - Wine-devel

Re: [jscript/tests] Prevent a crash on win9x
by Jacek Caban 10 Sep '08

10 Sep '08

Paul Vriens wrote: > Hi, > > This at least prevents the crash on win9x. There are still numerous > failures on > win9x but they are visible now. (I'm not so sure about returning > S_FALSE in this case btw). IMO it would be better to return error code and break whole test on win9x or find a way to do so before running script. If we wanted to support win9x we'd probably need to add objects representing each function, which is not worth the trouble. Thanks, Jacek

1 0

Re: Fixed an infite loop in winedbg: review wanted
by Eric Pouech 10 Sep '08

10 Sep '08

Florian Köberle a écrit : > Hello > > This patch address the infinite loop found at bug 15209: > http://bugs.winehq.org/show_bug.cgi?id=15209 > > At least one of the lines I moved in the loop doesn't make sense outsite: > count -= min(count, 256); > because the value of count will be always 0 after the loop. > > I also moved the following line which might be at the wrong position > too. About that I am not sure: > + fcp->Start += 256; > > Best regards, > Florian Koeberle > > ------------------------------------------------------------------------ > > looks a correct fix to me (moving both lines) A+ -- Eric Pouech "The problem with designing something completely foolproof is to underestimate the ingenuity of a complete idiot." (Douglas Adams)

1 0

Re: [PATCH 1/2] kernel32/test: Test import entries of a core windows DLLs (Revision 2)
by Paul Vriens 10 Sep '08

10 Sep '08

Roy Shea wrote: > This revision places the new test with the other loader tests in > loader.c, and for testing loads kernel32.dll rather than riched20.dll > > The prior version of this patch is available for reference at: > http://www.winehq.org/pipermail/wine-patches/2008-September/061226.html > --- > dlls/kernel32/tests/loader.c | 72 ++++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 72 insertions(+), 0 deletions(-) > > > ------------------------------------------------------------------------ > > Hi, Minor things : typo in "kerenl32_module" and shouldn't you free the library after the test? (Or maybe use GetModuleHandle, or does that defeat the test?) -- Cheers, Paul.

1 0

Patchwatcher security improvements
by Ambroz Bizjak 10 Sep '08

10 Sep '08

Hi, I've abandoned my chroot aproach to improving security in patchwatcher. Instead I've implemented the ability to run untrusted code as a user different than the one running patchwatcher. This is because creating a chroot where Wine could be compiled and tested proved to be too difficult and platform-dependent. I've also added external time limits for running untrusted code. This as a whole should help prevent individual patches from stalling the patch watching process. It very easy to set up. All you need is a low-privileged user (but enough to run the tests, e.g. audio, video groups) and an empty folder where you can write but this user can only read (not your home folder, it shouldn't have access there anyway). To use it, start with a clean patchwatcher and adjust the variables in the patchwatcher.sh, then run "patchwatcher.sh intialize". It will instruct you to run some commands as root (setuid the wrapper). Run initialize again and it should build wine and run baseline tests. Then you can test it by putting a patch in patches/ and issuing the try_one_patch command. To start watching use the continuous_build command. Patch is attached.

4 7

Re: gdi32: Don't crash when copying to an EMF from a NULL bitmap.
by Austin English 10 Sep '08

10 Sep '08

On Mon, Sep 8, 2008 at 3:03 PM, Austin English <austinenglish(a)gmail.com> wrote: > Based on an original patch by Mike McCormack. Rediffed against current > git. Testcase included, passes in WinXP. Fixes bug 4543. > Anything wrong with this one?

2 1

Re: Patchwatcher security improvements
by Ambroz Bizjak 10 Sep '08

10 Sep '08

Dan Kegel wrote: > So the slave can be in another real machine, another virtual machine, or running as another user; anything as long as it can get read/write access to its subdirectory of the shared directory. The problem with your design right now is that you want to run the slave in some isolated environment and expect it to be secure. The build slave itself is a mission-critical process and putting it in a quarantine to run together with untrusted code allows malicious patches to interfere with its operation. This means an attacker can just kill it from inside his patch, causing the whole patch building operation to fail, or corrupt the baseline tree, or send hundreds of fake emails through the slave interface. So I plan to run the build slave itself in a trusted environment, but make it quarantine individual build operations (similar to my previous design with user switching). This way the impact of an attack is highly limited - all it can theoretically do is fake his own patch results.

1 0

Re: Praise of sorts for Wine re Softmaker Office 2008
by Susan Cragin 10 Sep '08

10 Sep '08

>Dan Kegel wrote: >> http://forum.ubuntuusers.de/topic/softmaker-office-2008--eine-alternative--… >> has a review of an office suite named Softmaker Office 2008, >> available for Windows and now also Linux. The review says in part >> >> "Irritiert hat uns, daß die Windows-Version von Office 2008, die >> von Stick gestartet werden kann, unter Wine besser, schneller >> und stabiler läuft, wie die native Office 2008 für Linux. Wer also >> die Windows-Version legal erworben hat, kann hier Geld sparen." >> >> ("It irritated us that the Windows version runs better on Wine >> than the native Linux version. If you have the Windows version >> already, save your money and run it on Wine.") >> >> That's praise of sorts... although I hope it doesn't >> dissuade other vendors from doing native ports. >> - Dan >> > >I noticed the same type of thing when I was playing with DVD Shrink a couple >days ago. Ripping speed in DVD Shrink via Wine was significantly faster than on >Windows. What usually is a 2.5 hour process was about 1.5-1.75 hour process. > >I agree though, hopefully the software vendors will do native ports :). > >-Zac I just attended the convention for the National Verbatim Reporters Association in Savannah, where I demonstrated DNS 9.0 running on WINE with Ubuntu Studio (realtime kernel seems to make a slight difference). All the vendors were impressed, and one said he'd never seen DNS 9 run that fast, that accurately.

1 0

Re: [msi/tests] Create only one log file and delete it afterwards
by James Hawkins 10 Sep '08

10 Sep '08

On Wed, Sep 10, 2008 at 7:39 AM, Paul Vriens <paul.vriens.wine(a)gmail.com> wrote: > Hi, > > Instead of having 200 logfiles we now create one that we can delete > afterwards > as we know the name. This is not to speed up the tests but merely to get rid > of > the log files. > > We have to pass something for the log mode otherwise we would still have > multiple files. > > (It also looks like Wine is still appending to that one file which is not > the > same as on Windows). > > Changelog > Create only one log file and delete it afterwards > Why don't you just disable logging? MsiEnableLog(0, NULL, 0); -- James Hawkins

2 1

Re: Patchwatcher security improvements
by Ambroz Bizjak 10 Sep '08

10 Sep '08

Francois Gouget wrote: > This seems like an almost perfect task for a virtual machine: > * set up you virtual machine to taste > * take a snapshot > * to test a patch, fire up the virtual machine > * have it test the patch > * after the test or when it times out, revert it to the snapshot * rinse (done in the step above), repeat > > This could be done with VirtualBox, but maybe other alternatives based on Xen or KVM or some such would be better. The main issue I see with this is that the OpenGL / DirectSound tests will not run on the real hardware (as usual), but maybe a Xen-like approach could help there. > > It would also make it easy to test on FreeBSD / Solaris, at least if based on something like VirtualBox (not sure about the Xen-like > approaches). Seems like a good idea. I suggest only to use Linux as the host OS and run guests with KVM or VirtualBox, or even UML for Linux builds. Then we can run any OS that uses X as a guest and configure it to use the host's X display (through the virtual network). This should theoretically allow OpenGL through the XGL protocol, but VMGL can also be used if it doesn't work well. And sound hardware would be emulated by the VM software. I think I'll try getting a small Gentoo system to run in UML with a read-only root fs and make it boot as fast as possible. To try a patch, I would give it read access to the master Wine tree on the host, it would copy it to a writable temp folder and try it out. After it's finished or if the external timeout elapses, the UML process will be terminated and all of its writable storage will be reverted.

1 0

Re: explorer.exe: Implement SC_SCREENSAVE by starting xdg-screensaver
by Steven Edwards 10 Sep '08

10 Sep '08

On Wed, Sep 10, 2008 at 7:28 AM, Paul Chitescu <paulc(a)voip.null.ro> wrote: > Changelog: > explorer.exe: Implement SC_SCREENSAVE by starting xdg-screensaver with option > to lock the screen. > > The desktop window reacts to SC_SCREENSAVE by starting the screen saver, just > like it happens in Windows. No action is taken (and returns FALSE) if running > in a virtual desktop. > > Alternatively, a non-zero lparam is used to signal that the screen should be > locked. This cooperates with the user32.dll patch that implements > LockWorkStation(). This is an extension, there is no equivalent in Windows > but all the programs I've seen that trigger the system screen saver set > lparam=0. You forgot to attach the patch. Also you can send both patches together as one diff. There is no reason to split them up in this case. -- Steven Edwards "There is one thing stronger than all the armies in the world, and that is an idea whose time has come." - Victor Hugo

1 0