From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dushyant Behl Subject: [PATCH] extras/mini-os/tpmback.c : fix compilation error. Date: Mon, 21 Jul 2014 02:22:59 +0530 Message-ID: <1405889579-14705-1-git-send-email-myselfdushyantbehl@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "\"Xen Devel" Cc: Ian Campbell , Stefano Stabellini , Ian Jackson , Samuel Thibault , Daniel De Graaf , Dushyant Behl List-Id: xen-devel@lists.xenproject.org This patch is with respect to the following discussion on xen-devel - http://lists.xenproject.org/archives/html/xen-devel/2014-07/msg01991.html The file extras/mini-os/tpmback.c was failing compilation on certain compilers because of size mismatch between enum and int. Earlier the code used to read value of enum using %d format, which failed compilation on some compilers. Now the value is read into an actual int variable and then assigned to the enum. Signed-off-by:- Dushyant Behl --- extras/mini-os/tpmback.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/extras/mini-os/tpmback.c b/extras/mini-os/tpmback.c index 0601eb3..31da8d5 100644 --- a/extras/mini-os/tpmback.c +++ b/extras/mini-os/tpmback.c @@ -332,6 +332,7 @@ error_post_irq: * returns 0 on success and non-zero on error */ int tpmif_change_state(tpmif_t* tpmif, enum xenbus_state state) { + int tempst; char path[512]; char *value; char *err; @@ -347,11 +348,12 @@ int tpmif_change_state(tpmif_t* tpmif, enum xenbus_state state) free(err); return -1; } - if(sscanf(value, "%d", &readst) != 1) { + if(sscanf(value, "%d", &tempst) != 1) { TPMBACK_ERR("Non integer value (%s) in %s ??\n", value, path); free(value); return -1; } + readst = (enum xenbus_state) tempst; free(value); /* It's possible that the backend state got updated by hotplug or something else behind our back */ -- 1.9.1