quic: fix up coverity warning in quic/session.cc

[mhdawson]

• add CHECK around SocketAddress::New like we have in other places as suggested by Coverity scan

[mhdawson]

Coverity report:

02    case AF_INET: {
1303      auto ipv4 = preferredAddress->ipv4();
1304      if (ipv4.has_value()) {
1305        if (ipv4->address.empty() || ipv4->port == 0) return;
      	CID 318341 (#1 of 2): Unchecked return value (CHECKED_RETURN) [[select issue](https://scan9.scan.coverity.com/defectInstanceId=9078846&fileInstanceId=114681342&mergedDefectId=318341)]
1306        SocketAddress::New(AF_INET,
1307                           std::string(ipv4->address).c_str(),
1308                           ipv4->port,
1309                           &remote_address_);
1310        preferredAddress->Use(ipv4.value());
1311      }
1312      break;
1313    }
1314    case AF_INET6: {
1315      auto ipv6 = preferredAddress->ipv6();
    	3. Condition ipv6.has_value(), taking true branch.
1316      if (ipv6.has_value()) {
    	4. Condition ipv6->address.empty(), taking false branch.
    	5. Condition ipv6->port == 0, taking false branch.
1317        if (ipv6->address.empty() || ipv6->port == 0) return;
    	
CID 318341 (#2 of 2): Unchecked return value (CHECKED_RETURN)
6. check_return: Calling New without checking return value (as is done elsewhere 9 out of 11 times).
1318        SocketAddress::New(AF_INET,
1319                           std::string(ipv6->address).c_str(),
1320                           ipv6->port,
1321                           &remote_address_);
1322        preferredAddress->Use(ipv6.value());
1323      }
1324      br

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/54332/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/54375/

[mhdawson]

Landed in 971af4b