Sorry, but I don't believe your statement:
With your code, if the name entered exactly matches a line in names.txt, the function you had exits before it gets to the point that echoes:
When you find a match, you call exit which terminates the while loop, the function, and the shell script that called the function. When you don't find a match, you call exit which terminates the function and the script that called the function. I would think you would want something more like the following:

Code:

GetName() {
if [ $abc != "SO" ]
then    printf "Please enter the name: "
        read Name
        while read line
        do
                if [ "$Name" = "$line" ]
                then    return
                fi
        done < $dirUser/names.txt

        echo "$Name is not a valid name... Exiting"
        exit 1
fi
}

Without knowing the environment in which this function will be used, I would have expected the exit at the end of this function to be either:

Code:

exit 1

if you really want to exit the script calling this function indicating unsuccessful completion, or:

Code:

return 1

indicating that the function failed, but letting the rest of the script continue if it wants to based on the return value from the function. I used exit 1 in my rewrite to follow the statement in your last echo.

Furthermore, I didn't see any need for tempvar, since Name is always valid if $abc is not "SO" and this function returns to the calling program (after my changes) and can never be used if the name is not found (since your program exits).