summaryrefslogtreecommitdiff
path: root/zebra
diff options
context:
space:
mode:
authorgdt <gdt>2004-01-08 15:44:29 +0000
committergdt <gdt>2004-01-08 15:44:29 +0000
commitb27900b7c0c9b4105c8188e277b698b8e64c54cc (patch)
treecfce195dc0de508f263a6ad9b34359815d2c55d6 /zebra
parent9ccabd1cdb47cc932b6a43c9e28b6aac9f2a4a0c (diff)
2004-01-08 Greg Troxel <gdt@fnord.ir.bbn.com>
* kernel_socket.c (kernel_read): Use sockaddr_storage in buffer for reading kernel messages to ensure enough space (necessary on Solaris due to sockaddr_dl being large). Thanks to Sowmini Varadhan for help with this change.
Diffstat (limited to 'zebra')
-rw-r--r--zebra/ChangeLog7
-rw-r--r--zebra/kernel_socket.c23
2 files changed, 21 insertions, 9 deletions
diff --git a/zebra/ChangeLog b/zebra/ChangeLog
index 6cae04bd..8256c1cc 100644
--- a/zebra/ChangeLog
+++ b/zebra/ChangeLog
@@ -1,3 +1,10 @@
+2004-01-08 Greg Troxel <gdt@fnord.ir.bbn.com>
+
+ * kernel_socket.c (kernel_read): Use sockaddr_storage in buffer
+ for reading kernel messages to ensure enough space (necessary on
+ Solaris due to sockaddr_dl being large). Thanks to Sowmini
+ Varadhan for help with this change.
+
2004-01-06 Greg Troxel <gdt@t1.ir.bbn.com>
* rtadv.c (rtadv_send_packet): Change perror to zlog_err.
diff --git a/zebra/kernel_socket.c b/zebra/kernel_socket.c
index b8dfcc7d..9dd26157 100644
--- a/zebra/kernel_socket.c
+++ b/zebra/kernel_socket.c
@@ -849,10 +849,12 @@ kernel_read (struct thread *thread)
/*
* This must be big enough for any message the kernel might send.
- * The code previously used RTAX_MAX struct sockaddrs in all cases,
- * but now that sockaddrs are variable size, this doesn't work
- * (Solaris has 244 bytes of sdl_data!). For now, add a struct
- * sockaddr_dl to the case where it is used.
+ * Rather than determining how many sockaddrs of what size might be
+ * in each particular message, just use RTAX_MAX of sockaddr_storage
+ * for each. Note that the sockaddrs must be after each message
+ * definition, or rather after whichever happens to be the largest,
+ * since the buffer needs to be big enough for a message and the
+ * sockaddrs together.
*/
union
{
@@ -860,22 +862,21 @@ kernel_read (struct thread *thread)
struct
{
struct rt_msghdr rtm;
- struct sockaddr addr[RTAX_MAX];
+ struct sockaddr_storage addr[RTAX_MAX];
} r;
/* Interface information. */
struct
{
struct if_msghdr ifm;
- struct sockaddr_dl;
- struct sockaddr addr[RTAX_MAX-1];
+ struct sockaddr_storage addr[RTAX_MAX];
} im;
/* Interface address information. */
struct
{
struct ifa_msghdr ifa;
- struct sockaddr addr[RTAX_MAX];
+ struct sockaddr_storage addr[RTAX_MAX];
} ia;
#ifdef RTM_IFANNOUNCE
@@ -883,7 +884,7 @@ kernel_read (struct thread *thread)
struct
{
struct if_announcemsghdr ifan;
- struct sockaddr addr[RTAX_MAX];
+ struct sockaddr_storage addr[RTAX_MAX];
} ian;
#endif /* RTM_IFANNOUNCE */
@@ -908,6 +909,10 @@ kernel_read (struct thread *thread)
rtm = &buf.r.rtm;
+ /*
+ * Ensure that we didn't drop any data, so that processing routines
+ * can assume they have the whole message.
+ */
if (rtm->rtm_msglen != nbytes)
{
zlog_warn ("kernel_read: rtm->rtm_msglen %d, nbytes %d, type %d\n",