ReJeCtAll / naxsi

Automatically exported from code.google.com/p/naxsi
Other
0 stars 0 forks source link

Cannot compile naxsi #33

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Building with CentOS 5.8 64bits, gcc 4.1.2, nginx 1.2.0
Error:
cc1: warnings being treated as errors
src/naxsi/naxsi_utils.c: In function 
\u2018ngx_http_dummy_create_hashtables_n\u2019:
src/naxsi/naxsi_utils.c:635: warning: dereferencing type-punned pointer will 
break strict-aliasing rules

See attached file for compile details.

Original issue reported on code.google.com by ad...@yqed.com on 30 May 2012 at 11:44

Attachments:

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
The above compile error is with 0.45 version. Using 0.46-1, I get this error:
cc1: warnings being treated as errors
naxsi-0.46-1/naxsi_src/naxsi_runtime.c: In function 'gx_http_basestr_ruleset_n':
naxsi-0.46-1/naxsi_src/naxsi_runtime.c:929: error: 'nb_match' may be used 
uninitialized in this function

Please let me know what patch I should apply for 0.46-1. Thanks.

Original comment by ad...@yqed.com on 31 May 2012 at 12:20

GoogleCodeExporter commented 8 years ago
I solved the compile issues by removing the Werror cflag, inserted by default.
Still, it would nice if a clean fix is applied to the source code.

Original comment by ad...@yqed.com on 2 Jun 2012 at 7:11

GoogleCodeExporter commented 8 years ago
Hello, 
I made some change that might have fixed the issue.
Unfortunately, I don't have a CentOS to reproduce compilation issues, can you 
confirm if it's fixed, and if not, specify your compilation options ?

Thanks,

Original comment by ori...@gmail.com on 5 Jun 2012 at 11:57

GoogleCodeExporter commented 8 years ago
I'll compile it tonight and let you know. I would appreciate if you could look 
at issue 34, so I know of I should upgrade python. Once the rpm's are done, I 
will have them open to public. :-) 

Original comment by ad...@yqed.com on 5 Jun 2012 at 6:03

GoogleCodeExporter commented 8 years ago
Did the fix worked for the C source code ?

Original comment by ori...@gmail.com on 17 Jun 2012 at 12:12

GoogleCodeExporter commented 8 years ago

Original comment by ori...@gmail.com on 17 Jun 2012 at 12:13

GoogleCodeExporter commented 8 years ago
I actually compiled nginx is a different way with custom flags and everything 
compiled properly. I presume you made the changes into trunk? I have a script 
that I can download the trunk from Google, but for now the issue is fixed with 
original code. :)

Check your email, I replied to you.

Original comment by ad...@yqed.com on 17 Jun 2012 at 6:54