Skip to content

Feature: make log contain source address of unknown site #45

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.

Sign up for GitHub

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

Merged
dmuhamedagic merged 8 commits into from
Jun 29, 2016
Merged

Feature: make log contain source address of unknown site #45

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
9688c17
Refactor: mark no_leader properly as a constant pointer
jnpkrn Jun 16, 2016
add3382
Refactor: attr.c: merge if-else series into the switch statement
jnpkrn Jun 16, 2016
8ecd24f
Refactor: main: remove superfluous includes
jnpkrn Jun 16, 2016
ca0d94d
Fix typo: enought -> enough
jnpkrn Jun 16, 2016
e40d8b7
Comment: add_site: explain why inet_pton is endianess-save
jnpkrn Jun 16, 2016
95b8c17
Refactor: quasi-sort includes in transport.c
jnpkrn Jun 16, 2016
0a3f300
Low: remove redundant check after find_site_by_id
jnpkrn Jun 16, 2016
3b0bc3a
Feature: make log contain source address of unknown site
jnpkrn Jun 16, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Comment: add_site: explain why inet_pton is endianess-save
  • Loading branch information
@jnpkrn
jnpkrn committed Jun 16, 2016
commit e40d8b7a5e1fbb4d02f0d322d71207e6a562cf46
5 changes: 4 additions & 1 deletion src/config.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,10 @@ static int add_site(char *addr_string, int type)
* increase even more.
* Whether there'll be a collision in real-life, with 3 or 5 nodes, is
* another question ... but for now get the ID from the binary
* representation - that had *no* collisions up to 32.0.0.0. */
* representation - that had *no* collisions up to 32.0.0.0.
* Note that POSIX mandates inet_pton to arange the address pointed
* to by "dst" in network byte order, assuring little/big-endianess
* mutual compatibility. */
if (inet_pton(AF_INET,
site->addr_string,
&site->sa4.sin_addr) > 0) {
Expand Down