[bgpd] review 32-bit AS-path hotfix for 0.99.12
commit014b670e02cc1f38e8e4e786269fc1787412f9b7
authorDenis Ovsienko <pilot@etcnet.org>
Tue, 23 Jun 2009 17:10:45 +0000 (23 21:10 +0400)
committerDenis Ovsienko <pilot@etcnet.org>
Tue, 23 Jun 2009 17:10:45 +0000 (23 21:10 +0400)
treef0fa74e181d556d2d83d8db480fab097da72bafa
parentd13c3b4fcf802f904ef47ad82fdc9763fc704fdf
[bgpd] review 32-bit AS-path hotfix for 0.99.12

The patch by Chris Caputo, which was used to prepare 0.99.12
release, consists of three parts:

1. memory allocation fix itself
2. fix for warnings about constant variables
3. fix for printf format specs (%d was used instead of %u)

It was confirmed later, that:
a. a much simpler bugfix was available for memory allocation
b. committed version of the bugfix wasn't optimal CPU-wise

At this point I consider reasonable to revert the allocation
portion of that patch and to replace it with the shorter
version, which is:

-#define ASN_STR_LEN (5 + 1)
+#define ASN_STR_LEN (10 + 1)

Other two parts of Mr. Caputo's patch remain intact.
bgpd/bgp_aspath.c