From 93f8bfe0e33a1e7a9eb41c4a5290cfe43ed4c09c Mon Sep 17 00:00:00 2001 From: hunt Date: Tue, 18 Apr 2006 17:31:29 +0000 Subject: Fix accidentally commented out error message. --- tapset/ChangeLog | 4 ++++ tapset/conversions.stp | 9 +++++---- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/tapset/ChangeLog b/tapset/ChangeLog index a1cbf049..7d5ce509 100644 --- a/tapset/ChangeLog +++ b/tapset/ChangeLog @@ -1,3 +1,7 @@ +2006-04-18 Martin Hunt + + * conversions.stp (user_string): Reenable error message. + 2006-04-17 Martin Hunt * endian.stp (set_endian): New function. diff --git a/tapset/conversions.stp b/tapset/conversions.stp index c0e42cc5..d46f8c18 100644 --- a/tapset/conversions.stp +++ b/tapset/conversions.stp @@ -29,15 +29,16 @@ success: ; %} # NB: accessing user space is hazardous from certain kernel contexts. +# Errors should be returned when this is detected. function user_string:string (addr:long) %{ long rc = _stp_strncpy_from_user (THIS->__retvalue, (const char __user*) (uintptr_t) THIS->addr, MAXSTRINGLEN); if (rc < 0) { -// static char errmsg[40]; -// snprintf (errmsg, 40, "user string copy fault at 0x%p", -// (void *) (uintptr_t) THIS->addr); -// CONTEXT->last_error = errmsg; + static char errmsg[40]; + snprintf (errmsg, 40, "user string copy fault at 0x%p", + (void *) (uintptr_t) THIS->addr); + CONTEXT->last_error = errmsg; } %} -- cgit