[Openvas-devel] [Openvas-commits] r1103 - trunk/openvas-libnasl/nasl

Bernhard Herzog bh at intevation.de
Wed Aug 6 12:09:24 CEST 2008


scm-commit at wald.intevation.org writes:

> Fixed an issue with UDP socket close in nasl_close_socket
[...]
> --- trunk/openvas-libnasl/nasl/nasl_socket.c	2008-08-04 09:16:50 UTC (rev 1102)
> +++ trunk/openvas-libnasl/nasl/nasl_socket.c	2008-08-06 09:41:14 UTC (rev 1103)
> @@ -612,7 +612,7 @@
>   int e;
>   
>   soc = get_int_var_by_num(lexic, 0, -1);
> - if(soc <= 4)
> + if(soc < 4)
>  	{
>   	 nasl_perror(lexic, "close(): invalid argument\n");
>  	 return NULL;

This seems a bit strange.  What problem does that fix, exactly?  And
what's the relevance of the number 4 here?  Why don't the other NASL
functions that accept sockets as parameters contain similar checks?

   Bernhard

-- 
Bernhard Herzog                              Intevation GmbH, Osnabrück
Amtsgericht Osnabrück, HR B 18998             http://www.intevation.de/
Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagner


More information about the Openvas-devel mailing list