[asterisk-dev] [svn-commits] tilghman: branch tilghman/adaptive_realtime r115575 - in /team/tilghman/adapt...

[Michiel van Baak]

On 03:10, Fri 09 May 08, SVN commits to the Digium repositories wrote:
> Author: tilghman
> Date: Thu May  8 22:10:08 2008
> New Revision: 115575
> 
> URL: http://svn.digium.com/view/asterisk?view=rev&rev=115575
> Log:
> Add checks to existing realtime updates
> 
> Modified: team/tilghman/adaptive_realtime/apps/app_voicemail.c
> URL: http://svn.digium.com/view/asterisk/team/tilghman/adaptive_realtime/apps/app_voicemail.c?view=diff&rev=115575&r1=115574&r2=115575
> ==============================================================================
> --- team/tilghman/adaptive_realtime/apps/app_voicemail.c (original)
> +++ team/tilghman/adaptive_realtime/apps/app_voicemail.c Thu May  8 22:10:08 2008
> @@ -836,6 +836,9 @@
>  {
>  	int res;
>  	if (!ast_strlen_zero(vmu->uniqueid)) {
> +		if (strlen(password) > 10) {
> +			ast_realtime_require_field("voicemail", "password", RQ_CHAR, strlen(password), NULL);
> +		}

You only check for the password field in realtime if the password is
longer then 10 characters ? Why not check it every time?


-- 

Michiel van Baak
michiel at vanbaak.eu
http://michiel.vanbaak.eu
GnuPG key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x71C946BD

"Why is it drug addicts and computer aficionados are both called users?"