ftp: removes one use of atoi

Fixes only one small part of #3053

diff --git a/src/app-layer-ftp.c b/src/app-layer-ftp.c
index 6861e20e8adcaaec45fb891d2e674f2ce6b6ac32..08a25ede0db50d3086e26bbff632dacd662262b7 100644 (file)
--- a/src/app-layer-ftp.c
+++ b/src/app-layer-ftp.c
@@ -397,6 +397,8 @@ static uint16_t ftp_validate_port(int computed_port_value)
  */
 static uint16_t FTPGetV6PortNumber(uint8_t *input, uint32_t input_len)
 {
+    uint16_t res;
+
     uint8_t *ptr = memrchr(input, '|', input_len);
     if (ptr == NULL) {
         return 0;
@@ -410,7 +412,10 @@ static uint16_t FTPGetV6PortNumber(uint8_t *input, uint32_t input_len)
     if (ptr == NULL)
         return 0;
 
-    return ftp_validate_port(atoi((char *)ptr + 1));
+    if (ByteExtractStringUint16(&res, 10, 0, (const char *)ptr + 1) < 0) {
+        return 0;
+    }
+    return res;
 }
 
 /**