Hi,

A few style remarks about your patch :
- LF Line ending only
- Only C style comments, so no // …
- Indentation is 4 spaces.
- Add this setting to your .hgrc /mercurial.ini
[diff]
showfunc = True
- Do not remove existing blank lines .

Additional information here :
http://nginx.org/en/docs/contributing_changes.html

Regards,
Filipe

---

De : nginx-devel-bounces@nginx.org [mailto:nginx-devel-bounces@nginx.org] De la part de Sergey Brester
Envoyé : mercredi 22 avril 2015 09:46
À : nginx-devel@nginx.org
Objet : Fwd: Windows shmem fix: makes shared memory fully ASLR and DEP compliant (ea. cache zone, limit zone, etc.)

Hi,
enclosed you will find an attached changeset, that fixes a ASLR/DEP problem on windows platforms (example Win 7/2008 x64).
To find shared addr offset with ASLR, we have successful used the same resp. similar solution on various open source projects (ex.: postgresql etc.). Also nginx with suggested patch works fine over 5 months in production on several machines (Win-2k8-R2).
BTW(1): I have interest to fix a problem with multiple workers under windows (ex.: http://forum.nginx.org/read.php?2,188714,188714#msg-188714).
@Maxim Dounin: can you possibly give me more information, what you mean here, resp. what it currently depends on (see http://forum.nginx.org/read.php?2,188714,212568#msg-212568).
BTW(2): I would like to fix iocp under windows also. Thanks in advance for any information about.
P.S. speak fluently russian and german...


Regards,
Serg G. Brester (sebres)
________________________________________
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel