[MPlayer-dev-eng] [PATCH] fix for unconditional use of PATH_MAX
Robert Millan
zeratul2 at wanadoo.es
Thu May 1 14:35:01 CEST 2003
On Wed, Apr 30, 2003 at 07:39:27PM +0200, Fabian Franz wrote:
>
> I think we should stick to what we have now and just add, what I mentioned adn
> no it is no security risk, see Tobias Mail, about it.
please don't do that, it's a _bad_ solution. what if i run mplayer in
a directory longer than 512 bytes? then it will fail or not work as
expected.
if you hardcode PATH_MAX, you'll recieve bug reports from users telling
that mplayer fails or behaves unexpectedly.
if you don't want to stick with two code snippets (one for PATH_MAX and
other for non-PATH_MAX), then just use dynamic allocation as someone
suggested before. that will work for all systems.
--
Robert Millan
make: *** No rule to make target `war'. Stop.
Another world is possible - Just say no to genocide
More information about the MPlayer-dev-eng
mailing list