PEL: Remove extraneous return statements

UserHeader::validate() had some extra return statements
that aren't needed.  It is desired for the function to run
all the way through even on a validation failure.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I9b99970179cd7c84a0201b7d67e39b0bee209beb
diff --git a/extensions/openpower-pels/user_header.cpp b/extensions/openpower-pels/user_header.cpp
index 117ade1..917d578 100644
--- a/extensions/openpower-pels/user_header.cpp
+++ b/extensions/openpower-pels/user_header.cpp
@@ -46,7 +46,6 @@
         log<level::ERR>("Invalid user header section ID",
                         entry("ID=0x%X", header().id));
         failed = true;
-        return;
     }
 
     if (header().version != userHeaderVersion)
@@ -54,7 +53,6 @@
         log<level::ERR>("Invalid user header version",
                         entry("VERSION=0x%X", header().version));
         failed = true;
-        return;
     }
 
     _valid = (failed) ? false : true;