Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

possible crash send sending notifications #259

Open
ntadas opened this issue Mar 8, 2017 · 0 comments
Open

possible crash send sending notifications #259

ntadas opened this issue Mar 8, 2017 · 0 comments

Comments

@ntadas
Copy link
Contributor

ntadas commented Mar 8, 2017

Hi,

In session.c line 1572 https://github.com/CESNET/libnetconf/blob/master/src/session.c#L1572
the strlen (text) should be checked before accessing the content of the text.
currently I'm having a crash (very sporadic) and I was able to nail it until here (not sure the content of "text" when the crash happens).

#2 0x0f230968 in crashHandler (signo=<optimized out>, sigInfo=<optimized out>, ptr=<optimized out>) at ../base/src/crashHandler.c:62 #3 <signal handler called> #4 0x4821bfac in write () from /lib/libpthread.so.0 #5 0x0fb7b850 in nc_session_send (session=0x26d59ef0, msg=0x2a29a0c0) at src/session.c:1572 #6 0x0fb7ea78 in nc_session_send_reply (session=0x26d59ef0, rpc=0x2a29a330, reply=0x268f7ef0) at src/session.c:2945

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant