Re: [PATCH] Limit the number of screens Xvfb will attempt to allocate memory for

2011-11-25 Thread Jamey Sharp
Sure, that makes sense. Reviewed-by: Jamey Sharp ja...@minilop.net On 11/23/11, Alan Coopersmith alan.coopersm...@oracle.com wrote: Commit f9e3a2955d2ca7 removing the MAXSCREEN limit left the screen number too unlimited, and allowed any positive int for a screen number: Xvfb :1 -screen

[PATCH] Limit the number of screens Xvfb will attempt to allocate memory for

2011-11-23 Thread Alan Coopersmith
Commit f9e3a2955d2ca7 removing the MAXSCREEN limit left the screen number too unlimited, and allowed any positive int for a screen number: Xvfb :1 -screen 2147483647 1024x1024x8 Fatal server error: Not enough memory for screen 2147483647 Found by Parfait 0.3.7: Error: Integer overflow (CWE 190)