]> git.proxmox.com Git - mirror_frr.git/commit
bgpd: fix aspath parsing
authorVincent JARDIN <vincent.jardin@6wind.com>
Mon, 9 Oct 2017 07:43:26 +0000 (09:43 +0200)
committerVincent JARDIN <vincent.jardin@6wind.com>
Mon, 9 Oct 2017 07:48:53 +0000 (09:48 +0200)
commitb42d80dd9a8ba9e40b2001bd6d4e9baf6cb7dc10
treea675f794e337b3eb4cd2be4ed9323069ebfa8f74
parente4002056b8b7de774223b91e897da6ce0f15860a
bgpd: fix aspath parsing

clang provides a notice about it that this p++ is useless,
because ++ would be done after the return.

From code review, I understand that p shall be incremented
for each token that is parsed from the buf. So let's keep
this intent.

Note that this commit is changing the behaviour of the source
code since from now p++ will be returned instead of p.
However, it does not hurt since the only consumer
just free() the aspath if it is parsed as as_token_unknown.
Let's be safe with a proper execution flow from now.

PS:
C reminders:

int f7(void) {
  int j = 7;

  return ++j; // return 8
}

int f8(void) {
  int j = 7;

  return j++; // return 7
}

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>
bgpd/bgp_aspath.c