diff options
| author |   Florin Coras <fcoras@cisco.com> | 2019-05-09 12:08:06 -0700 |
|---|---|---|
| committer |   Dave Barach <openvpp@barachs.net> | 2019-05-09 20:36:14 +0000 |
| commit | 404b8a36e737e2911ca8874363a4e87cb707e5f9 (patch) | |
| tree | f1388afdb67309754074e98f72dfc9afb645e0c4 /src/vnet/session/segment_manager.c | |
| parent | 7d31ab2a5cc2124ddd973ac5dfac2219619f345a (diff) | |
session: segment manager fixes
- fix segment size rounding for 4GB segments
- fix initialization of first segment size
- cleanup fifo segment info retrieval
Change-Id: I5ebf20f71ea797087653e7e76fa2e37b2686ec40
Signed-off-by: Florin Coras <fcoras@cisco.com>
Diffstat (limited to 'src/vnet/session/segment_manager.c')
| -rw-r--r-- | src/vnet/session/segment_manager.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/vnet/session/segment_manager.c b/src/vnet/session/segment_manager.c index 6213cd517f6..cbec493e55f 100644 --- a/src/vnet/session/segment_manager.c +++ b/src/vnet/session/segment_manager.c @@ -114,7 +114,9 @@ segment_manager_add_segment (segment_manager_t * sm, u32 segment_size) */ segment_size = segment_size ? segment_size : props->add_segment_size; page_size = clib_mem_get_page_size (); - segment_size = (segment_size + page_size - 1) & ~(page_size - 1); + /* Protect against segment size u32 wrap */ + segment_size = clib_max (segment_size + page_size - 1, segment_size); + segment_size = segment_size & ~(page_size - 1); if (props->segment_type != SSVM_SEGMENT_PRIVATE) { seg_name = format (0, "%d-%d%c", getpid (), smm->seg_name_counter++, 0); |