bug 702 - SLB: Failed to allocate memory resources |AS-12626|Epassport||

Review Request #487 — Created Oct. 9, 2024 and updated

tanya
APV10
rel_apv_10_7, rel_apv_10_7_2
702
kdutta, kevin.poh, satyendra, timlai, wli

when we execute "show slb policy all" command the memory error shows up in "show log b b" output. Note: For the error to show up slb needs to have NO HTTPS virtual servieces configured.
This is just a false alarm due to some wrong error checking after malloc. we pass 0 size to malloc so we get nothing back.
Other slb show policy functions already have this check, so I added it here as well.
If we enable englog there is englog msg that points to show_slb_policy_doh_kern_helper() line 9225 slb_vs_policy_kern()

in englog mode
debug on
debug file ...
debug module slb all
debug module cli all

then run
"show slb policy all"
before the fix the englog error was showing up and after the fix it does not.

tanya
Review request changed

Description:

~  

when we execute "show slb policy all" command the memory error shows up in show log b b output. Note: For the error to show up slb need to have NO HTTPS virtual servieces configured.

~   This is just false alarm due to some wrong error checking after malloc. we pass 0 size to malloc so we get nothing back.
~   Other slb show policy functions already have this check so I added it here as well.
~   If enabled end log there is eng log that points to show_slb_policy_doh_kern_helper() line 9225 slb_vs_policy_kern()

  ~

when we execute "show slb policy all" command the memory error shows up in "show log b b" output. Note: For the error to show up slb needs to have NO HTTPS virtual servieces configured.

  ~ This is just a false alarm due to some wrong error checking after malloc. we pass 0 size to malloc so we get nothing back.
  ~ Other slb show policy functions already have this check, so I added it here as well.
  ~ If we enable englog there is englog msg that points to show_slb_policy_doh_kern_helper() line 9225 slb_vs_policy_kern()

kevin.poh
  1. Ship It!
  2. 
      
timlai
  1. Ship It!
  2. 
      
Loading...