Revert GCC-12 workarounds that don't seem to be required anymore

Author: skmcgrail

crypto/x509/v3_utl.c

@@ -1288,25 +1288,9 @@ static int ipv6_from_asc(uint8_t v6[16], const char *in) {
 
   // Format the result.
   if (v6stat.zero_pos >= 0) {
-    // Due to the gcc-12 workaround parsing is broke for the IPv6 address
-    // `::` which would have been previously supported and working.
-    // This variant is checked earlier, so total here can't be anything
-    // other then zero if the count was 3.
-    // if(v6stat.zero_cnt == 3) {
-    //   OPENSSL_memset(v6, 0, 16);
-    //   return 1;
-    // }
-
     // Copy initial part
     OPENSSL_memcpy(v6, v6stat.tmp, v6stat.zero_pos);
-
     // Zero middle
-    // This condition is to suppress gcc-12 warning.
-    // https://github.com/aws/aws-lc/issues/487
-    // if (v6stat.zero_pos > v6stat.total) {
-    //     // This should not happen.
-    //     return 0;
-    // }
     OPENSSL_memset(v6 + v6stat.zero_pos, 0, 16 - v6stat.total);
     // Copy final part
     if (v6stat.total != v6stat.zero_pos) {