diff options
author | Edwin Mons <edwin.mons@isode.com> | 2018-11-09 08:51:10 (GMT) |
---|---|---|
committer | Edwin Mons <edwin.mons@isode.com> | 2018-11-09 10:05:12 (GMT) |
commit | a45eed2d826f60a20dfd1f4f06df37d1f83263f8 (patch) | |
tree | 7593509f9b89630ff70d5c9e1d14966435a4ab8a /Swiften/Network/NATPortMapping.h | |
parent | 644db9fce94670e610db46bdd28eb9d1e658a798 (diff) | |
download | swift-a45eed2d826f60a20dfd1f4f06df37d1f83263f8.zip swift-a45eed2d826f60a20dfd1f4f06df37d1f83263f8.tar.bz2 |
Store NATPortMapping lifetime as uint32_t
The NAT-PMP RFC specifies lifetime as an unsigned, 32 bit integer, so
using this internally makes sense. This also removes the need for casts
in interface code.
A slight bug in the NATPMPInterface implementation to remove a mapping
has been addressed.
Test-Information:
Unit tests pass on macOS 10.13
Change-Id: I06687fd2735a8706f339f85d6c957ad7aadaaf02
Diffstat (limited to 'Swiften/Network/NATPortMapping.h')
-rw-r--r-- | Swiften/Network/NATPortMapping.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Swiften/Network/NATPortMapping.h b/Swiften/Network/NATPortMapping.h index b68052d..bf0fb1c 100644 --- a/Swiften/Network/NATPortMapping.h +++ b/Swiften/Network/NATPortMapping.h @@ -7,3 +7,3 @@ /* - * Copyright (c) 2015 Isode Limited. + * Copyright (c) 2015-2018 Isode Limited. * All rights reserved. @@ -25,3 +25,3 @@ namespace Swift { - NATPortMapping(unsigned short localPort, unsigned short publicPort, Protocol protocol = TCP, int leaseInSeconds = 60 * 60 * 24) : + NATPortMapping(unsigned short localPort, unsigned short publicPort, Protocol protocol = TCP, uint32_t leaseInSeconds = 60 * 60 * 24) : publicPort(publicPort), localPort(localPort), protocol(protocol), leaseInSeconds(leaseInSeconds) { @@ -42,3 +42,3 @@ namespace Swift { - int getLeaseInSeconds() const { + uint32_t getLeaseInSeconds() const { return leaseInSeconds; @@ -50,3 +50,3 @@ namespace Swift { Protocol protocol; - int leaseInSeconds; + uint32_t leaseInSeconds; }; |