Latest AV code

jim jim.brown at rsmas.miami.edu
Thu Dec 28 00:31:50 UTC 2017


Trever,
You missed one change to 'if (!ok)' and one change to 'if (ok)'.
Jim

On 12/27/2017 7:03 PM, Trever L. Adams wrote:
> @@ -635,8 +647,8 @@ static virusfilter_action virusfilter_do_infected_file_action(
>   		} else {
>   			DBG_DEBUG("quarantine: Creating "
>   			      "directory %s\n", temp_path);
> -			if (quarantine_create_dir(handle,
> -			    config, temp_path) == false)
> +			ok = quarantine_create_dir(handle, config, temp_path);
> +			if (ok == false)

On 12/27/2017 7:03 PM, Trever L. Adams wrote:
> @@ -112,10 +113,11 @@ static virusfilter_result virusfilter_fsav_scan_init(
>   		DBG_DEBUG("fsavd: Checking if connection is alive\n");
>   
>   		/* FIXME: I don't know the correct PING command format... */
> -		if (virusfilter_io_writefl_readl(io_h, &reply, "PING") ==
> -		    true)
> +		ok = virusfilter_io_writefl_readl(io_h, &reply, "PING");
> +		if (ok == true)



More information about the samba-technical mailing list