Fix small bugs

* src/androidterm.c (android_handle_ime_event): Pacify compiler
warnings.
* src/textconv.c (really_set_composing_text)
(handle_pending_conversion_events, get_extracted_text): Fix
reentrancy problems and uses of uninitialized values.

1 files changed, 16 insertions, 3 deletions

diff --git a/src/textconv.c b/src/textconv.c
index b62ed7d1365..a39748457d3 100644
--- a/src/textconv.c
+++ b/src/textconv.c
@@ -583,6 +583,8 @@ really_set_composing_text (struct frame *f, ptrdiff_t position,
                    Fpoint (), Qnil);
       Fset_marker_insertion_type (f->conversion.compose_region_end,
                                   Qt);
+
+      start = position;
     }
   else
     {
@@ -875,14 +877,25 @@ handle_pending_conversion_events (void)
 
       /* Test if F has any outstanding conversion events.  Then
          process them in bottom to up order.  */
-      for (action = f->conversion.actions; action; action = next)
+      while (true)
         {
           /* Redisplay in between if there is more than one
-             action.  */
+             action.
+
+             This can read input.  This function must be reentrant
+             here.  */
 
           if (handled)
             redisplay ();
 
+          /* Reload action.  */
+          action = f->conversion.actions;
+
+          /* If there are no more actions, break.  */
+
+          if (!action)
+            break;
+
           /* Unlink this action.  */
           next = action->next;
           f->conversion.actions = next;
@@ -1134,7 +1147,7 @@ get_extracted_text (struct frame *f, ptrdiff_t n,
 
   /* Detect overflow.  */
 
-  if (!(start <= PT <= end))
+  if (!(start <= PT && PT <= end))
     goto finish;
 
   /* Convert the character positions to byte positions.  */