Changeset dfb2144 in rtems-libbsd

Timestamp:

05/22/23 07:36:46 (4 months ago)

Author:

Christian Mauderer <christian.mauderer@…>

Branches:

master

Children:

af0fcc3

Parents:

16be3a7

git-author:

Christian Mauderer <christian.mauderer@…> (05/22/23 07:36:46)

git-committer:

Christian Mauderer <christian.mauderer@…> (05/31/23 06:21:12)

Message:

ipsec-tools: Fix copying fd_set prior to select

The racoon session code copies an fd_set from one variable into another
prior to calling select. That works well for simple structures.

In libbsd we have to allocate fd_sets instead of using fixed structures
to avoid a problem with file numbers bigger than FD_SETSIZE. The simple
assignment didn't work in that case.

This patch makes sure that a memcpy is used instead.

Update #4913

File:

• ipsec-tools/src/racoon/session.c (modified) (2 diffs)

ipsec-tools/src/racoon/session.c

@@ -216 +216 @@
         FD_ZERO(&preset_mask);
 #else /* __rtems__ */
+        size_t allocated_mask_size = sizeof(fd_set) *
+            howmany(rtems_libio_number_iops, sizeof(fd_set) * 8);
         allocated_preset_mask = calloc(sizeof(fd_set),
             howmany(rtems_libio_number_iops, sizeof(fd_set) * 8));
@@ -353 +355 @@
                 /* schedular can change select() mask, so we reset
                  * the working copy here */
+#ifndef __rtems__
                 active_mask = preset_mask;
+#else /* __rtems__ */
+                memcpy(allocated_active_mask, allocated_preset_mask,
+                    allocated_mask_size);
+#endif /* __rtems__ */
 
                 error = select(nfds + 1, &active_mask, NULL, NULL, timeout);