[Serusers] Shoot my logic full of holes....

[sip]

Okay... I'm wondering if this would work as a quick hack for checking call
blocking (and making it rational). 

In order to use AVPOps, and not rewrite a module, I've created a table called
callblock, which looks exactly like the usr_preferences table (basically an
AVP table), but has no primary keys (as I want to be able to add multiple
values for the same username/domain/attribute). 

In my code, I then have a block that looks like:

            if(avp_db_load("$ruri/username","s:callblkna/callblock"))
            {
                if(avp_check("s:callblkna","eq/$from/gi"))
                {
                    sl_send_reply("603", "Call denied");
                    break;
                };
            };
            if(avp_db_load("$ruri/username","s:callblkvm/callblock"))
            {
                if(avp_check("s:callblkvm","eq/$from/gi"))
                {
                    route(4);
                    break;
                };
            };

If it's a callblkna, it then sends an error reply. If it's a callblkvm, it
does a route(4) (which is straight to voicemail). 

Anyone see a glaring hole in this logic here? It seems to work, but I'd hate
to get down the road and discover that I overlooked something glaringly
obvious (which I often do). 


N.