Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 707078 - obsolete (and uninitialized) variable ignore_schedd_limit is being used in place of ignore_submitter_limit
obsolete (and uninitialized) variable ignore_schedd_limit is being used in pl...
Status: CLOSED ERRATA
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: condor (Show other bugs)
2.0
Unspecified Unspecified
high Severity medium
: 2.0.1
: ---
Assigned To: Erik Erlandson
Luigi Toscano
:
Depends On:
Blocks: 723887
  Show dependency treegraph
 
Reported: 2011-05-23 18:34 EDT by Erik Erlandson
Modified: 2011-09-07 12:41 EDT (History)
5 users (show)

See Also:
Fixed In Version: condor-7.6.2-0.1
Doc Type: Bug Fix
Doc Text:
Cause: Inner negotiation loop called with an obsolete and uninitialized stack variable. Consequence: Depending on value of uninitialized stack memory, there is potential for either incorrectly ignoring submitter limits when limits should be respected, or conversely obeying submitter limits when they are expected to be ignored. Fix: Obsolete variable was removed and proper parameter to the inner negotiation loop was restored. Result: Guaranteed proper behavior regarding submitter limits is restored.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2011-09-07 12:41:18 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2011:1249 normal SHIPPED_LIVE Moderate: Red Hat Enterprise MRG Grid 2.0 security, bug fix and enhancement update 2011-09-07 12:40:45 EDT

  None (edit)
Description Erik Erlandson 2011-05-23 18:34:19 EDT
Description of problem:
the negotiate() function is being called with obsolete variable ignore_schedd_limit, which is uninitialized.   It should be using ignore_submitter_limit.


How reproducible:
ignore_schedd_limit is uninitialized -- how reproducible is undetermined.

Steps to Reproduce:
NOTE: Should only ignore submitter limits on 1st spin of pie, and if considering preemption.

Two possible repros:
1) false-pos (ignore submitter limits when shouldn't):
ignoring limits when consider-preemption is false
ignoring limits on spin-2 or higher

2) false-neg (should ignore submitter limits, but does not)
obeying the limits on first spin, and when consider-preemption is true.
Comment 1 Erik Erlandson 2011-05-23 19:31:28 EDT
Upstream fix (7.7/master)
https://condor-wiki.cs.wisc.edu/index.cgi/chngview?cn=21957

Attached to upstream ticket:
https://condor-wiki.cs.wisc.edu/index.cgi/tktview?tn=2054,0
Comment 2 Erik Erlandson 2011-05-25 18:10:32 EDT
Fixed upstream on V7_6-branch.
Comment 6 Luigi Toscano 2011-07-21 14:01:56 EDT
The variable ignore_schedd_limit is not used anymore when the negotiate() function is invoked, and the correct ignore_submitter_limit is used instead. ignore_schedd_limit itself has been removed.

Verified by source code inspection, condor-7.6.3-0.2 for RHEL5 and RHEL6.
Comment 7 Erik Erlandson 2011-07-25 18:30:01 EDT
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Cause:
Inner negotiation loop called with an obsolete and uninitialized stack variable.

Consequence:
Depending on value of uninitialized stack memory, there is potential for either incorrectly ignoring submitter limits when limits should be respected, or conversely obeying submitter limits when they are expected to be ignored.

Fix:
Obsolete variable was removed and proper parameter to the inner negotiation loop was restored.

Result:
Guaranteed proper behavior regarding submitter limits is restored.
Comment 8 errata-xmlrpc 2011-09-07 12:41:18 EDT
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2011-1249.html

Note You need to log in before you can comment on or make changes to this bug.