Merge bitcoin#22169: p2p, rpc, fuzz: various tiny follow-ups

c274574 p2p, rpc, fuzz: various tiny follow-ups (Jon Atack)

Pull request description:

  - p2p: pass `Span` by value per bitcoin#22143 (comment) as a follow-up to 8be56f0
  - rpc: remove duplicate `CAddress` constructor per bitcoin#22043 (comment)
  - fuzz: rename 3 fuzz targets changed in eba9a94 back to their original names per bitcoin-core/qa-assets#63 (comment)

ACKs for top commit:
  MarcoFalke:
    cr ACK c274574
  practicalswift:
    cr ACK c274574: patch looks correct
  jarolrod:
    ACK  c274574

Tree-SHA512: 3672b210d30b3a91f3a6455005e4d3cb1f89621820c417c645d24b06e53459440122a1f75758e0e04c3d04eff9d6f88ef62865216aa3e42301c6df783f7c0b4a

Author: fanquake

src/netaddress.cpp

@@ -558,7 +558,7 @@ static std::string IPv6ToString(Span<const uint8_t> a)
     // clang-format on
 }
 
-static std::string OnionToString(const Span<const uint8_t>& addr)
+static std::string OnionToString(Span<const uint8_t> addr)
 {
     uint8_t checksum[torv3::CHECKSUM_LEN];
     torv3::Checksum(addr, checksum);

src/test/fuzz/deserialize.cpp

@@ -212,15 +212,15 @@ FUZZ_TARGET_DESERIALIZE(coins_deserialize, {
     Coin coin;
     DeserializeFromFuzzingInput(buffer, coin);
 })
-FUZZ_TARGET_DESERIALIZE(net_address_deserialize, {
+FUZZ_TARGET_DESERIALIZE(netaddr_deserialize, {
     CNetAddr na;
     DeserializeFromFuzzingInput(buffer, na);
     if (na.IsAddrV1Compatible()) {
         AssertEqualAfterSerializeDeserialize(na);
     }
     AssertEqualAfterSerializeDeserialize(na, INIT_PROTO_VERSION | ADDRV2_FORMAT);
 })
-FUZZ_TARGET_DESERIALIZE(net_service_deserialize, {
+FUZZ_TARGET_DESERIALIZE(service_deserialize, {
     CService s;
     DeserializeFromFuzzingInput(buffer, s);
     if (s.IsAddrV1Compatible()) {