Subject: [P2PSIP] minor ambiguities in draft-ietf-p2psip-base-18

From: stéphane bryant <[email-masked]>
Date: Fri, 26 Aug 2011 08:21:44 +0200

Hello,

I have found what seems to be two typos or ambiguities
in draft-ietf-p2psip-base-18:

* In section 8 (TURN Server Usage), the TurnServer
  structure is defined as:

	struct {
          uint8                   iteration;
          IpAddressAndPort        server_address;
        } TurnServer;

However, IpAddressAndPort, is not defined anywhere,
what is defined instead, in 5.3.1.1 (Common Definitions)
is IpAddressPort

* In section 5.5.1.1 (Request Definition, for Attach),
  in the description of the IceCandidate structure it
  says:
  <<
rel_addr_port
      corresponds to the rel-addr and rel-port productions.  Only
      present for type "relay".

  However, in the structure itself, we have:
<<
	select (type){
            case host:
              ;          /* Nothing */
            case srflx:
            case prflx:
            case relay:
              IpAddressPort     rel_addr_port;
          };

The rel_addr_port is therefore present for the types
srflx, prflx, and relay.

Regards
stephane bryant