diff options
| author | Ross Burton <ross.burton@intel.com> | 2018-08-13 18:20:54 +0100 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-08-14 11:36:31 +0100 |
| commit | a316be88cddffe448faa242536d1e6d8279554fb (patch) | |
| tree | 685bd66afe48be133bc1e589b7c55407c5d02b01 /meta/classes/license.bbclass | |
| parent | 4b2513defabcc3659d129137f82f9a73dec2f472 (diff) | |
| download | poky-a316be88cddffe448faa242536d1e6d8279554fb.tar.gz | |
classes: sanity-check LIC_FILES_CHKSUM
We assume that LIC_FILES_CHKSUM is a file: URI but don't actually verify this,
which can lead to problems if you have a URI that resolves to a path of / as
Bitbake will then dutifully checksum / recursively.
[ YOCTO #12883 ]
(From OE-Core rev: e2b8a3d5a10868f9c0dec8d7b9f5f89fdd100fc8)
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/license.bbclass')
| -rw-r--r-- | meta/classes/license.bbclass | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass index 2ac15a1bc8..4cf7f074fc 100644 --- a/meta/classes/license.bbclass +++ b/meta/classes/license.bbclass | |||
| @@ -205,7 +205,9 @@ def find_license_files(d): | |||
| 205 | 205 | ||
| 206 | for url in lic_files.split(): | 206 | for url in lic_files.split(): |
| 207 | try: | 207 | try: |
| 208 | (type, host, path, user, pswd, parm) = bb.fetch.decodeurl(url) | 208 | (method, host, path, user, pswd, parm) = bb.fetch.decodeurl(url) |
| 209 | if method != "file" or not path: | ||
| 210 | raise bb.fetch.MalformedUrl() | ||
| 209 | except bb.fetch.MalformedUrl: | 211 | except bb.fetch.MalformedUrl: |
| 210 | bb.fatal("%s: LIC_FILES_CHKSUM contains an invalid URL: %s" % (d.getVar('PF'), url)) | 212 | bb.fatal("%s: LIC_FILES_CHKSUM contains an invalid URL: %s" % (d.getVar('PF'), url)) |
| 211 | # We want the license filename and path | 213 | # We want the license filename and path |
