Skip to content

Conversation

@cmcfarlen
Copy link
Contributor

The fds_limit extern is causing the issues that global variables typically do. This PR cleans that up by using a function to access the value in a much more reasonable place in the dependency chain.

@cmcfarlen cmcfarlen self-assigned this Aug 26, 2023
@cmcfarlen cmcfarlen added this to the 10.0.0 milestone Aug 26, 2023
@cmcfarlen cmcfarlen merged commit 313406a into apache:master Aug 28, 2023
@cmcfarlen cmcfarlen deleted the untangle-fds-limit branch August 28, 2023 22:45
cmcfarlen added a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
* commit '01409a61bab49c81cf422cdb29a94dcd07bf3c37': (31 commits)
  cmake: add more executables (apache#10284)
  Fix a crash due to bad disks (apache#10287)
  ts_lua_transform: input_reader may be used uninitialized (apache#10297)
  CID 1508887 fix parent_select plugin stat Time of check. (apache#10260)
  Make remap API compatibility checks consistent. (apache#10253)
  remove deprecated 'NATIVE' AIO mode (apache#10269)
  Fix connect failure count (apache#10294)
  Add version range up to CMake 3.27 (apache#10259)
  untangle fds_limit global (apache#10288)
  replace net_config_poll_timeout with EThread::default_wait_interval_ms (apache#10289)
  suppress coverity false positive on race condition (apache#10278)
  initialize variable (apache#10277)
  Check the inet_pton return value for IP reputations (apache#10267)
  Fix missing virtual destructor. (apache#10293)
  tools/jtest: use @SWOC_LIBS@ (apache#10292)
  Fix m4 quoting on configure error messages. (apache#10258)
  build_h3_tools: explicitly chmod the destination directory (apache#10274)
  Fix CID 1508979: unchecked return value (apache#10281)
  Fix CID 1508970: Uninitialized scalar variable (apache#10282)
  Fix CID 1508992, 1508976, 1508972: uninitialized scalar variables (apache#10275)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants