On Sun 2006-09-03 23:53:47, Tim Dijkstra wrote:
> On Sun, 3 Sep 2006 22:49:57 +0200
> Pavel Machek <[EMAIL PROTECTED]> wrote:
> 
> > On Sun 2006-09-03 21:06:12, Tim Dijkstra wrote:
> > > Hi,
> > > 
> > > If I can count, than PARAM_NO is of by one;)
> > > 
> > > GEN_PARAM is defined as 8, which is correct because there is no
> > > #ifdef SPLASH_CONFIG anymore. SPLASH_PARAM still adds one to
> > > PARAM_NO though. 
> > > 
> > > This crashes s2disk/resume when there's an error in the config file.
> > 
> > Will not your patch break splash? Ugh, I do not see how param system
> > works :-(.
> 
> No it will not break. SPLASH_PARAM is only used to increment PARAM_NO
> by one. PARAM_NO is used to limit the loop through all options while 
> parsing the config file.
> 
> grep SPLASH_PARAM suspend-cvs/*
> suspend-cvs/swsusp.h:#define SPLASH_PARAM       1
> suspend-cvs/swsusp.h:#define PARAM_NO   (GEN_PARAM + COMPRESS_PARAM + 
> ENCRYPT_PARAM + SPLASH_PARAM)
>

Ok, patch applied, thanks!
                                                                        Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Suspend-devel mailing list
Suspend-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/suspend-devel

Reply via email to