[jira] [Comment Edited] (PROTON-2546) Leak of raw connection pn_event

2022-05-23 Thread Ken Giusti (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17541003#comment-17541003
 ] 

Ken Giusti edited comment on PROTON-2546 at 5/23/22 3:33 PM:
-

How to reproduce:

Build skupper-router from the main branch in Debug mode 
(-DCMAKE_BUILD_TYPE=Debug) with -DRUNTIME_CHECK=asan

Set env vars to filter out known leak suppressions (use the correct path to the 
tests/*.supp files, and the lsan.supp is in BUILD directory!) :

export ASAN_OPTIONS="disable_coredump=0 detect_odr_violation=0 
strict_string_checks=1 detect_stack_use_after_return=1 
check_initialization_order=1 strict_init_order=1 detect_invalid_pointer_pairs=2 
suppressions={*}/home/kgiusti/work/skupper/skupper-router/tests/asan.supp{*}"

export LSAN_OPTIONS="disable_coredump=0 
suppressions={*}/home/kgiusti/work/skupper/skupper-router/BUILD/tests/lsan.supp{*}"

 

Run the skupper router against the attached config file:

skrouterd -c proton-2546.conf &

In a separate window, run curl against the router "curl 
[http://127.0.0.1:8000|http://127.0.0.1:8000/];

while curl is running, kill the router: $ kill 

That should trigger a leak of the pn_raw_connection.

 


was (Author: kgiusti):
How to reproduce:

Build skupper-router from the main branch in Debug mode 
(-DCMAKE_BUILD_TYPE=Debug) with -DRUNTIME_CHECK=asan

Set env vars to filter out known leak suppressions (use the correct path to the 
tests/*.supp files, and the lsan.supp is in BUILD directory!) :

export ASAN_OPTIONS="disable_coredump=0 detect_odr_violation=0 
strict_string_checks=1 detect_stack_use_after_return=1 
check_initialization_order=1 strict_init_order=1 detect_invalid_pointer_pairs=2 
suppressions={*}/home/kgiusti/work/skupper/skupper-router/tests/asan.supp{*}"

export LSAN_OPTIONS="disable_coredump=0 
suppressions={*}/home/kgiusti/work/skupper/skupper-router/BUILD/tests/lsan.supp{*}"

 

Run the skupper router against the attached config file:

skrouterd proton-2546.conf &

In a separate window, run curl against the router "curl http://127.0.0.1:8000;

while curl is running, kill the router: $ kill 

That should trigger a leak of the pn_raw_connection.

 

> Leak of raw connection pn_event
> ---
>
> Key: PROTON-2546
> URL: https://issues.apache.org/jira/browse/PROTON-2546
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-c
>Affects Versions: proton-c-0.38.0
>Reporter: Ken Giusti
>Assignee: Andrew Stitcher
>Priority: Major
> Attachments: proton-2546.conf
>
>
> See: 
> [https://github.com/skupperproject/skupper-router/issues/469|https://github.com/skupperproject/skupper-router/issues/469?]
> This leak appeared on main about five days ago:
> https://github.com/skupperproject/skupper-router/runs/6489086985?check_suite_focus=true#step:26:1757
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2546) Leak of raw connection pn_event

2022-05-23 Thread Ken Giusti (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17541003#comment-17541003
 ] 

Ken Giusti commented on PROTON-2546:


How to reproduce:

Build skupper-router from the main branch in Debug mode 
(-DCMAKE_BUILD_TYPE=Debug) with -DRUNTIME_CHECK=asan

Set env vars to filter out known leak suppressions (use the correct path to the 
tests/*.supp files, and the lsan.supp is in BUILD directory!) :

export ASAN_OPTIONS="disable_coredump=0 detect_odr_violation=0 
strict_string_checks=1 detect_stack_use_after_return=1 
check_initialization_order=1 strict_init_order=1 detect_invalid_pointer_pairs=2 
suppressions={*}/home/kgiusti/work/skupper/skupper-router/tests/asan.supp{*}"

export LSAN_OPTIONS="disable_coredump=0 
suppressions={*}/home/kgiusti/work/skupper/skupper-router/BUILD/tests/lsan.supp{*}"

 

Run the skupper router against the attached config file:

skrouterd proton-2546.conf &

In a separate window, run curl against the router "curl http://127.0.0.1:8000;

while curl is running, kill the router: $ kill 

That should trigger a leak of the pn_raw_connection.

 

> Leak of raw connection pn_event
> ---
>
> Key: PROTON-2546
> URL: https://issues.apache.org/jira/browse/PROTON-2546
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-c
>Affects Versions: proton-c-0.38.0
>Reporter: Ken Giusti
>Assignee: Andrew Stitcher
>Priority: Major
> Attachments: proton-2546.conf
>
>
> See: 
> [https://github.com/skupperproject/skupper-router/issues/469|https://github.com/skupperproject/skupper-router/issues/469?]
> This leak appeared on main about five days ago:
> https://github.com/skupperproject/skupper-router/runs/6489086985?check_suite_focus=true#step:26:1757
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (PROTON-2546) Leak of raw connection pn_event

2022-05-23 Thread Ken Giusti (Jira)


 [ 
https://issues.apache.org/jira/browse/PROTON-2546?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ken Giusti updated PROTON-2546:
---
Attachment: proton-2546.conf

> Leak of raw connection pn_event
> ---
>
> Key: PROTON-2546
> URL: https://issues.apache.org/jira/browse/PROTON-2546
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-c
>Affects Versions: proton-c-0.38.0
>Reporter: Ken Giusti
>Assignee: Andrew Stitcher
>Priority: Major
> Attachments: proton-2546.conf
>
>
> See: 
> [https://github.com/skupperproject/skupper-router/issues/469|https://github.com/skupperproject/skupper-router/issues/469?]
> This leak appeared on main about five days ago:
> https://github.com/skupperproject/skupper-router/runs/6489086985?check_suite_focus=true#step:26:1757
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2546) Leak of raw connection pn_event

2022-05-23 Thread Ken Giusti (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17540999#comment-17540999
 ] 

Ken Giusti commented on PROTON-2546:


I've been debugging this a bit and think that the pn_raw_connection is being 
leaked.

-
Suppressions used:
  count      bytes template
      *2       3072 ^pn_raw_connection$*
      6        384 sys_mutex
    390     408014 /libpython3.*.so
-

SUMMARY: AddressSanitizer: 896 byte(s) leaked in 18 allocation(s).

 

I can reproduce these leaks by shutting down the skupper-router while there is 
at least one active pn_raw_connection. I shut down the router by sending a 
SIGTERM to it via kill.  In this case the router does not attempt to call 
pn_raw_connection_close() and wait for the disconnect.  Instead the shutdown 
code calls pn_proactor_free() and expects any outstanding raw connection memory 
to be freed by that.

Is this assumption correct?

If no raw connections are present at shutdown these leaks do not occur.

 

> Leak of raw connection pn_event
> ---
>
> Key: PROTON-2546
> URL: https://issues.apache.org/jira/browse/PROTON-2546
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-c
>Affects Versions: proton-c-0.38.0
>Reporter: Ken Giusti
>Assignee: Andrew Stitcher
>Priority: Major
>
> See: 
> [https://github.com/skupperproject/skupper-router/issues/469|https://github.com/skupperproject/skupper-router/issues/469?]
> This leak appeared on main about five days ago:
> https://github.com/skupperproject/skupper-router/runs/6489086985?check_suite_focus=true#step:26:1757
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org