]> sourceware.org Git - systemtap.git/commitdiff
Remove redundant assert_is_myproc from utf functions
authorJosh Stone <jistone@redhat.com>
Wed, 29 May 2013 19:53:59 +0000 (12:53 -0700)
committerJosh Stone <jistone@redhat.com>
Wed, 29 May 2013 19:53:59 +0000 (12:53 -0700)
* tapset/uconversions.stp (user_string_utf16, user_string_utf32): These
  are marked "/* myproc-unprivileged */", so the translator will insert
  the necessary assert_is_myproc() already.  Having a redundant check
  doesn't actually hurt, except it adds confusion by implying that other
  similar functions are in error for not having this too.

tapset/uconversions.stp

index 7683a2016db1b1550a4046451036b84d63c16391..d8e1d6b46f5fa8b93f1869e1324458658309b2bc 100644 (file)
@@ -203,8 +203,6 @@ function user_string_utf32:string (addr:long) %{ /* pure */ /* myproc-unprivileg
   uint32_t c32, *source = (uint32_t*)(intptr_t)STAP_ARG_addr;
   char *destination = STAP_RETVALUE;
 
-  assert_is_myproc();
-
   *destination = '\0';
   while (len > 1 && (c32 = uread(source))) {
     if ((rc = _stp_convert_utf32(destination, len, c32)) <= 0) {
@@ -255,8 +253,6 @@ function user_string_utf16:string (addr:long) %{ /* pure */ /* myproc-unprivileg
   uint16_t c16low, *source = (uint16_t*)(intptr_t)STAP_ARG_addr;
   char *destination = STAP_RETVALUE;
 
-  assert_is_myproc();
-
   *destination = '\0';
   while (len > 1 && (c32 = uread(source))) {
     /* Check for a UTF-16 high surrogate, then its low pair, and combine them.
This page took 0.028131 seconds and 5 git commands to generate.