[Pkg-dspam-misc] Patch to include a directory with files with parameters.

Matthijs Mohlmann matthijs at cacholong.nl
Thu Nov 10 16:25:51 UTC 2005


José Luis Tallón wrote:
> Matthijs Mohlmann wrote:
> 
> 
>>Hi,
>>
>>I managed to fix it. Please review the attached patch. And after a few
>>hours I'll upload it to the svn repository.
>>
>>[snip]
>> 
>>
> 
> Looks allright, except that you really don't check the error to see if
> it really was a file which we tried to open as opposed to a dir.
> 
I don't know if I fixed that fully, but there is now some more checking.

> Regarding the final '\n', i suggest this instead:
> ---
> // look for a linefeed at the end...
> char* ptr=strrchr(path,'\n');
> // ... and nuke it if found
> if( ptr )*ptr='\0';
> 
ah, didn't know there was a function strrchr (know your string operations)

> --------
> then you can avoid unconditionally using "strlen(path)-1" everywhere.
> strncpy is good :-)
> Looks fine -- sorry, don't have the time to test it right now.
> 

Thanks for checking the patch, I'll put it in svn later tonight.
Including postinst and postrm scripts for mysql and postgresql.

Regards,

Matthijs Mohlmann



More information about the Pkg-dspam-misc mailing list