Not to mention that if($data{value} eq  "" || null) isn't going to do anything meaningful. It parses out as if ((($data{'value'} eq '') or 'null')). What you would want to do would be to check if it's defined or not.

And given this is just a small smippet out of context I don't even want to get into what horrors of recursive calling could be brought on with that &adduser; line judging from the comment (aside from the fact that you really don't want to call a sub like that (ampersand with no parens) unless you know why you're doing it that way; just call mysub()).