buglwIP - A Lightweight TCP/IP stack - Bugs: bug #60443, Build error when NO_SYS==1 and...

 
 

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

bug #60443: Build error when NO_SYS==1 and SYS_LIGHTWEIGHT_PROT==1

Submitter:  Eric Koldeweij <eric_koldeweij>
Submitted:  Thu 22 Apr 2021 07:51:50 PM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  Compiler Warning Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Planned Release:  None
lwIP version:  git head

Thu 22 Apr 2021 07:55:50 PM UTC, comment #1: 

Patch is not correct. Accidentally used #ifdef instead of #if.
Please use patch.txt instead of patch. My apologies.

(file #51308)

Eric Koldeweij <eric_koldeweij>
Thu 22 Apr 2021 07:51:50 PM UTC, original submission:  

Small issue. Bug submitted for head but present in all releases I worked with since at least v2.0.2.

A missing definition for sys_prot_t is the culprit:
========
In file included from src/core/init.c:42:0:
./src/include/lwip/sys.h:507:1: error: unknown type name 'sys_prot_t'; did you mean 'sys_mbox_t'?
 sys_prot_t sys_arch_protect(void);
 ^~~~~~~~~~
 sys_mbox_t
./src/include/lwip/sys.h:508:25: error: unknown type name 'sys_prot_t'; did you mean 'sys_mbox_t'?
 void sys_arch_unprotect(sys_prot_t pval);
                         ^~~~~~~~~~
                         sys_mbox_t
========

This only occurs if in lwipopts.h both NO_SYS and SYS_LIGHTWEIGHT_PROT are used. If either or both are zero the error does not occur.

Attached is a small patch to fix this issue

Eric Koldeweij <eric_koldeweij>

 

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

Attached Files
file #51308:  patch.txt added by eric_koldeweij (384B - text/plain)
file #51307:  patch added by eric_koldeweij (387B - application/octet-stream - This patch defines sys_prot_t in the NO_SYS section when SYS_LIGHTWEIGHT_PROT != 0)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by eric_koldeweij (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.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-04-22 eric_koldeweij Attached File- Added patch.txt, #51308
    2021-04-22 eric_koldeweij Attached File- Added patch, #51307

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code