Comment 28 for bug 1634265

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat-translator (master)

Reviewed: https://review.openstack.org/420564
Committed: https://git.openstack.org/cgit/openstack/heat-translator/commit/?id=3cf56881413adf39963401279a84a3cc3fce19b9
Submitter: Jenkins
Branch: master

commit 3cf56881413adf39963401279a84a3cc3fce19b9
Author: Luong Anh Tuan <email address hidden>
Date: Mon Jan 16 15:20:45 2017 +0700

    Replace yaml.load() with yaml.safe_load()

    Avoid dangerous file parsing and object serialization libraries.
    yaml.load is the obvious function to use but it is dangerous[1]
    Because yaml.load return Python object may be dangerous if you
    receive a YAML document from an untrusted source such as the
    Internet. The function yaml.safe_load limits this ability to
    simple Python objects like integers or lists.

    In addition, Bandit flags yaml.load() as security risk so replace
    all occurrences with yaml.safe_load(). Thus I replace yaml.load()
    with yaml.safe_load()

    [1]https://security.openstack.org/guidelines/dg_avoid-dangerous-input-parsing-libraries.html

    Change-Id: Ifaecff145e91f72911ae05ea274a4977c56212c7
    Closes-Bug: #1634265