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

This pull request fix follow issues #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NTMan
Copy link

@NTMan NTMan commented Jan 6, 2018

  1. Flushing buffers when logger terminated by user. Without this fix you always see 0 sized log file when test your logger by command "echo test > /dev/udp/127.0.0.1/6666" and than terminate program by "ctrl-c" for see result.

  2. Fixing segfault when recieving more than two messages in one second from different ports. For reproducing issue you may just run "echo test > /dev/udp/127.0.0.1/6666 && echo test > /dev/udp/127.0.0.1/6666".

  3. Fixing segfault when unnable open new logfile. This issue you can ordinary reproduce if launch "echo test > /dev/udp/127.0.0.1/6666" more than 1024 times.

…ated by user. Without this fix you always see 0 sized log file when test your logger by command "echo test > /dev/udp/127.0.0.1/6666" and than terminate program by "ctrl-c" for see result. 2) Fixing segfault when recieving more than two messages in one second from different ports. For reproducing issue you may just run "echo test > /dev/udp/127.0.0.1/6666 && echo test > /dev/udp/127.0.0.1/6666". 3) Fixing segfault when unnable open new logfile. This issue you can ordinary reproduce if launch "echo test > /dev/udp/127.0.0.1/6666" more than 1024 times.
@NTMan
Copy link
Author

NTMan commented Jan 6, 2018

This version is obsolete you must update from here:
https://osdn.net/projects/akari/scm/svn/tree/head/branches/udplogger/
And forget about my pull request.

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

Successfully merging this pull request may close these issues.

1 participant