Re: [CIL users] integer overflow in merger using cilly

2012-07-03 Thread Gabriel Kerneis
On Tue, Jul 03, 2012 at 04:49:58PM +0200, Oliver Schwahn wrote: > I tested the patch and it seems to work fine! I encountered no > problems at all. Committed, thanks. -- Gabriel -- Live Security Virtual Conference Exclus

Re: [CIL users] integer overflow in merger using cilly

2012-07-03 Thread Oliver Schwahn
Hi, I tested the patch and it seems to work fine! I encountered no problems at all. Cheers, Oliver -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape

Re: [CIL users] integer overflow in merger using cilly

2012-07-03 Thread Gabriel Kerneis
On Tue, Jul 03, 2012 at 03:42:33PM +0200, Oliver Schwahn wrote: > >Please, could you try the attached patch (completely untested) and tell me > >if it > >works? > > Hm... I cannot find the patch. Maybe you forgot to attach it? My mistake, here it is. -- Gabriel diff --git a/lib/Cilly.pm.in b/li

Re: [CIL users] integer overflow in merger using cilly

2012-07-03 Thread Oliver Schwahn
Hi Gabriel, > I'm afraid commenting out wouldn't work (well not sure, but it could be > stripped > too early in preprocessing passes), but I just realised we could use a string > instead of an integer? I think using a string here seems to be the best option. > Please, could you try the attached

Re: [CIL users] integer overflow in merger using cilly

2012-07-03 Thread Gabriel Kerneis
Oliver, On Tue, Jul 03, 2012 at 10:02:52AM +0200, Oliver Schwahn wrote: > Another way to fix the issue on 32-bit may also be to comment out > the pragma directive since it's only used by cilly for bookkeeping > -- as far as i can tell. If the pragma is not needed by CIL, > commenting it out may be

Re: [CIL users] integer overflow in merger using cilly

2012-07-03 Thread Oliver Schwahn
Hi! >> A quick workaround is to set the system clock to a time before 2004. > Or to use a 64-bit architecture (yielding max_int = 2^63-1). You are right, using a 64-bit architecture is, of course, also a workaround but I have to stick to my 32-bit system :-( >> I think this issue can be fixed ei

Re: [CIL users] integer overflow in merger using cilly

2012-07-02 Thread Gabriel Kerneis
On Mon, Jul 02, 2012 at 03:36:02PM +0200, Oliver Schwahn wrote: > I just tried to use the merger (together with cilly) but i noticed > that it fails with error message: > Unimplemented: something.c:1: Int constant too large: 1341229737 Thanks for the report and detailed analysis. > A quick workar

[CIL users] integer overflow in merger using cilly

2012-07-02 Thread Oliver Schwahn
Hi all, I just tried to use the merger (together with cilly) but i noticed that it fails with error message: Unimplemented: something.c:1: Int constant too large: 1341229737 It turns out that the problem is the "#pragma merger (...)"which is written by cilly as first line of the merged output