buglwIP - A Lightweight TCP/IP stack - Bugs: bug #62159, Http doesn't compile with...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #62159: Http doesn't compile with LWIP_DEBUG and LWIP_TCP_KEEPALIVE

Submitter:  Jarno Malmari <jmalmari>
Submitted:  Wed 09 Mar 2022 11:24:47 AM UTC
   
 
Category:  apps Severity:  3 - Normal
Item Group:  Compiler Warning Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Planned Release:  None
lwIP version:  git head

Wed 17 May 2023 02:56:15 PM UTC, comment #2: 

Any chance of fixing this?

Peter Harper <peterharperuk>
Fri 10 Mar 2023 12:59:13 PM UTC, comment #1: 

Attached a patch.

(file #54468)

Jarno Malmari <jmalmari>
Wed 09 Mar 2022 11:24:47 AM UTC, original submission:  

In apps/http/altcp_proxyconnect.c, defining both of the above definitions makes altcp_proxyconnect_functions initialization list out of sync with its type altcp_functions.

Compilation failed because it was trying to initialize altcp_keepalive_disable_fn with altcp_default_dbg_get_tcp_state.

Not defining LWIP_DEBUG compiles but initializes altcp_keepalive_disable_fn and altcp_keepalive_enable_fn as nulls. I wonder if that's ok.

% git describe
STABLE-2_1_0_RELEASE-542-g215dec65

% git grep 'set(LWIP_VERSION_' src/Filelists.cmake                                                                                                                  
src/Filelists.cmake:set(LWIP_VERSION_MAJOR    "2")
src/Filelists.cmake:set(LWIP_VERSION_MINOR    "2")
src/Filelists.cmake:set(LWIP_VERSION_REVISION "0")
src/Filelists.cmake:set(LWIP_VERSION_RC       "LWIP_RC_DEVELOPMENT")

Jarno Malmari <jmalmari>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #54468:  0001-Fix-compilation-with-LWIP_DEBUG.patch added by jmalmari (831B - text/x-patch - Added missing initializations)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by peterharperuk (Posted a comment)
  • -email is unavailable- added by jmalmari (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-03-10 jmalmari Attached File- Added 0001-Fix-compilation-with-LWIP_DEBUG.patch, #54468

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code