aboutsummaryrefslogtreecommitdiffstats
path: root/recipes-support/spice
AgeCommit message (Collapse)Author
2018-03-12Revert "spice: Fix compile errors from gcc 7 with spice 0.12.4"Bruce Ashfield
This reverts commit 7f8de36343df74ba7aacc2ccb61e3b78d9b90ac0.
2018-03-08spice: Fix compile errors from gcc 7 with spice 0.12.4Jason Wessel
../../git/server/red_parse_qxl.c:367:18: error: 'BITMAP_FMT_IS_RGB' defined but not used [-Werror=unused-const-variable=] static const int BITMAP_FMT_IS_RGB[] = {0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1}; ^~~~~~~~~~~~~~~~~ ../../git/server/inputs_channel.c: In function 'inputs_channel_handle_parsed': ../../git/server/inputs_channel.c:317:38: error: this statement may fall through [-Werror=implicit-fallthrough=] case SPICE_MSGC_INPUTS_KEY_DOWN: { ^ ../../git/server/inputs_channel.c:324:5: note: here case SPICE_MSGC_INPUTS_KEY_UP: { ^~~~ ../../git/server/reds.c: In function 'vdi_port_read_one_msg_from_device': ../../git/server/reds.c:797:31: error: this statement may fall through [-Werror=implicit-fallthrough=] state->read_state = VDI_PORT_READ_STATE_GET_BUFF; ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../../git/server/reds.c:798:9: note: here case VDI_PORT_READ_STATE_GET_BUFF: { ^~~~ ../../git/server/reds.c:807:31: error: this statement may fall through [-Werror=implicit-fallthrough=] state->read_state = VDI_PORT_READ_STATE_READ_DATA; ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../../git/server/reds.c:809:9: note: here case VDI_PORT_READ_STATE_READ_DATA: ^~~~ This patch just adds the fallthrough comments which are already checked in upstream along with the removal of the static variable which is not used. The newer versions of spice already have this fixed, so there is no need to further upstream this patch. It gets dropped in the future. Signed-off-by: Jason Wessel <jason.wessel@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2017-09-29spice: fix CVE-2017-7506Yi Zhao
CVE-2017-7506: spice versions though 0.13 are vulnerable to out-of-bounds memory access when processing specially crafted messages from authenticated attacker to the spice server resulting into crash and/or server memory leak. Reference: https://nvd.nist.gov/vuln/detail/CVE-2017-7506 Patches from: https://cgit.freedesktop.org/spice/spice/commit/?h=0.12&id=f1e7ec03e26ab6b8ca9b7ec060846a5b706a963d https://cgit.freedesktop.org/spice/spice/commit/?h=0.12&id=ec6229c79abe05d731953df5f7e9a05ec9f6df79 https://cgit.freedesktop.org/spice/spice/commit/?h=0.12&id=a957a90baf2c62d31f3547e56bba7d0e812d2331 Signed-off-by: Yi Zhao <yi.zhao@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2017-08-24spice: fix builds with gcc 7.xMark Asselstine
There is an upstream fix for this issue but it doesn't apply cleanly and is rather simple so rather than backport it and turn it into something unrecognizeable I simply created my own version of it. This fixes builds which fail with: | ../../git/server/reds.c: In function 'vdi_port_read_one_msg_from_device': | ../../git/server/reds.c:797:31: error: this statement may fall through [-Werror=implicit-fallthrough=] | state->read_state = VDI_PORT_READ_STATE_GET_BUFF; | ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ../../git/server/reds.c:798:9: note: here | case VDI_PORT_READ_STATE_GET_BUFF: { | ^~~~ Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2017-07-25spice: set COMPATIBLE_HOSTChen Qi
Set COMPATIBLE_HOST to restrict building only for x86 archs. The configure script clearly requires x86 archs. Message is as below. Only x86 and x86-64 are supported Signed-off-by: Chen Qi <Qi.Chen@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2017-07-25spice: Fix build failuresHe Zhe
Backport 3cb746329ea4846bd9c65e0198e69423379b6f62 "Use PRI macros in printf to keep compatibility between 32/64bit system" from upstream to fix the following errors: server/jpeg_encoder.c:639:17: error: format '%lu' expects argument of type 'long unsigned int', but argument 12 has type 'uint64_t {aka long long unsigned int}' [-Werror=format=] server/mjpeg_encoder.c:639:17: error: format '%lu' expects argument of type 'long unsigned int', but argument 13 has type 'uint64_t {aka long long unsigned int}' [-Werror=format=] server/mjpeg_encoder.c:706:21: error: format '%lu' expects argument of type 'long unsigned int', but argument 6 has type 'uint64_t {aka long long unsigned int}' [-Werror=format=] server/mjpeg_encoder.c:706:21: error: format '%lu' expects argument of type 'long unsigned int', but argument 7 has type 'uint64_t {aka long long unsigned int}' [-Werror=format=] server/mjpeg_encoder.c:936:75: error: integer overflow in expression [-Werror=overflow] Signed-off-by: He Zhe <zhe.he@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2017-06-29spice: Fix compilation errorHe Zhe
| ../../git/server/red_parse_qxl.c:367:18: error: 'BITMAP_FMT_IS_RGB' defined but not used [-Werror=unused-const-variable=] | static const int BITMAP_FMT_IS_RGB[] = {0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1}; Signed-off-by: He Zhe <zhe.he@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2016-08-31spice: Add missing license to LICENSEMark Asselstine
There is a new QA check warning which has been added which flags situation where a subpackage includes a license which is not included in the recipe's LICENSE string. In this case we are seeing: WARNING: .../meta-cloud-services/recipes-support/spice/spice_git.bb: LICENSE_lib32-spice-protocol includes licenses (BSD) that are not listed in LICENSE Adding the missing license to the LICENSE string. Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2016-05-31spice: add PACKAGECONFIG for xineramaJackie Huang
spice auto detect Xinerama library to enable the feature, but it causes error when using sstate between different builds, e.g. previous build have xinerama but the later one doesn't, then it fails on the later one with: "Package 'xinerama', required by 'spice-server', not found" Add PACKAGECONFIG for xinerama to ensure we have deterministic builds. Signed-off-by: Jackie Huang <jackie.huang@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2015-09-10spice: add SRCREV_FORMAT to recipeJoe Slater
Create a composite source revision. Failure to do so can cause parsing errors. Signed-off-by: Joe Slater <jslater@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2014-05-13spice: allow separated src and build dirsMark Asselstine
Some minor fixes around generated .c and .h files were required to allow separated src and build dirs. Without these builds will fail and you would have to build in the src dir. Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2014-03-26spice: updates to work with latest autotoolsMark Asselstine
We were getting several errors during configure related to autoreconf. We need to create an empty m4 directory as well as enable the subdir-objects option to allow configure to complete successfully. There were indications that this second issue was fixed upstream but even using the latest git trees I am seeing this failure. Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
2014-03-25meta-cloud-services: introduce spiceBruce Ashfield
A common requirement for many cloud systems is remote console access. Adding spice to the common layer allows a rich console environment to be provided. Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>