diff options
| -rw-r--r-- | src/decrypt-verify.c | 2 | ||||
| -rw-r--r-- | src/decrypt.c | 2 | ||||
| -rw-r--r-- | src/edit.c | 6 | ||||
| -rw-r--r-- | src/encrypt-sign.c | 2 | ||||
| -rw-r--r-- | src/encrypt.c | 2 | ||||
| -rw-r--r-- | src/engine-backend.h | 2 | ||||
| -rw-r--r-- | src/engine-gpg.c | 62 | ||||
| -rw-r--r-- | src/engine.c | 6 | ||||
| -rw-r--r-- | src/engine.h | 3 | ||||
| -rw-r--r-- | src/genkey.c | 8 | ||||
| -rw-r--r-- | src/keysign.c | 2 | ||||
| -rw-r--r-- | src/passwd.c | 2 | ||||
| -rw-r--r-- | src/sign.c | 2 | 
13 files changed, 23 insertions, 78 deletions
| diff --git a/src/decrypt-verify.c b/src/decrypt-verify.c index 17f79acd..ce4a7a9b 100644 --- a/src/decrypt-verify.c +++ b/src/decrypt-verify.c @@ -74,7 +74,7 @@ decrypt_verify_start (gpgme_ctx_t ctx, int synchronous,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -	(ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +	(ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)  	return err;      } diff --git a/src/decrypt.c b/src/decrypt.c index ecd9c144..7dbc6fd6 100644 --- a/src/decrypt.c +++ b/src/decrypt.c @@ -510,7 +510,7 @@ _gpgme_decrypt_start (gpgme_ctx_t ctx, int synchronous,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -	(ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +	(ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)  	return err;      } @@ -139,8 +139,7 @@ interact_start (gpgme_ctx_t ctx, int synchronous, gpgme_key_t key,    opd->fnc_old = NULL;    opd->fnc_value = fnc_value; -  err = _gpgme_engine_set_command_handler (ctx->engine, command_handler, -					   ctx, out); +  err = _gpgme_engine_set_command_handler (ctx->engine, command_handler, ctx);    if (err)      return err; @@ -219,8 +218,7 @@ edit_start (gpgme_ctx_t ctx, int synchronous, int type, gpgme_key_t key,    opd->fnc_old = fnc;    opd->fnc_value = fnc_value; -  err = _gpgme_engine_set_command_handler (ctx->engine, command_handler, -					   ctx, out); +  err = _gpgme_engine_set_command_handler (ctx->engine, command_handler, ctx);    if (err)      return err; diff --git a/src/encrypt-sign.c b/src/encrypt-sign.c index 4db46e25..cc34fbd5 100644 --- a/src/encrypt-sign.c +++ b/src/encrypt-sign.c @@ -93,7 +93,7 @@ encrypt_sign_start (gpgme_ctx_t ctx, int synchronous, gpgme_key_t recp[],    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -	(ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +	(ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)  	return err;      } diff --git a/src/encrypt.c b/src/encrypt.c index 2318497e..a27a53ac 100644 --- a/src/encrypt.c +++ b/src/encrypt.c @@ -242,7 +242,7 @@ encrypt_start (gpgme_ctx_t ctx, int synchronous, gpgme_key_t recp[],      {        /* Symmetric encryption requires a passphrase.  */        err = _gpgme_engine_set_command_handler -	(ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +	(ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)  	return err;      } diff --git a/src/engine-backend.h b/src/engine-backend.h index f6926662..4f33da1c 100644 --- a/src/engine-backend.h +++ b/src/engine-backend.h @@ -55,7 +55,7 @@ struct engine_ops  			      void *fnc_value);    gpgme_error_t (*set_command_handler) (void *engine,  					engine_command_handler_t fnc, -					void *fnc_value, gpgme_data_t data); +					void *fnc_value);    gpgme_error_t (*set_colon_line_handler) (void *engine,  					   engine_colon_line_handler_t fnc,  					   void *fnc_value); diff --git a/src/engine-gpg.c b/src/engine-gpg.c index e9558202..43d49fe6 100644 --- a/src/engine-gpg.c +++ b/src/engine-gpg.c @@ -135,10 +135,6 @@ struct engine_gpg      char *keyword;       /* what has been requested (malloced) */      engine_command_handler_t fnc;      void *fnc_value; -    /* The kludges never end.  This is used to couple command handlers -       with output data in edit key mode.  */ -    gpgme_data_t linked_data; -    int linked_idx;    } cmd;    struct gpgme_io_cbs io_cbs; @@ -502,8 +498,6 @@ gpg_new (void **engine, const char *file_name, const char *home_dir,    gpg->colon.fd[1] = -1;    gpg->cmd.fd = -1;    gpg->cmd.idx = -1; -  gpg->cmd.linked_data = NULL; -  gpg->cmd.linked_idx = -1;    /* Allocate the read buffer for the status pipe.  */    gpg->status.bufsize = 1024; @@ -792,14 +786,14 @@ command_handler (void *opaque, int fd) -/* The Fnc will be called to get a value for one of the commands with -   a key KEY.  If the Code passed to FNC is 0, the function may release -   resources associated with the returned value from another call.  To -   match such a second call to a first call, the returned value from -   the first call is passed as keyword.  */ +/* The FNC will be called to get a value for one of the commands with + * a key KEY.  If the code passed to FNC is 0, the function may + * release resources associated with the returned value from another + * call.  To match such a second call to a first call, the returned + * value from the first call is passed as keyword.  */  static gpgme_error_t  gpg_set_command_handler (void *engine, engine_command_handler_t fnc, -			 void *fnc_value, gpgme_data_t linked_data) +			 void *fnc_value)  {    engine_gpg_t gpg = engine;    gpgme_error_t rc; @@ -818,7 +812,6 @@ gpg_set_command_handler (void *engine, engine_command_handler_t fnc,    gpg->cmd.fnc = fnc;    gpg->cmd.cb_data = (void *) &gpg->cmd;    gpg->cmd.fnc_value = fnc_value; -  gpg->cmd.linked_data = linked_data;    gpg->cmd.used = 1;    return 0;  } @@ -1076,11 +1069,6 @@ build_argv (engine_gpg_t gpg, const char *pgmname)  		  assert (gpg->cmd.idx == -1);  		  gpg->cmd.idx = datac;  		} -	      else if (gpg->cmd.linked_data == a->data) -		{ -		  assert (gpg->cmd.linked_idx == -1); -		  gpg->cmd.linked_idx = datac; -		}  	    }  	  fd_data_map[datac].data = a->data; @@ -1267,44 +1255,6 @@ read_status (engine_gpg_t gpg)  			  if (err)  			    return err;                          } - -		      if (r == GPGME_STATUS_END_STREAM) -			{ -			  if (gpg->cmd.used) -			    { -			      /* Before we can actually add the -				 command fd, we might have to flush -				 the linked output data pipe.  */ -			      if (gpg->cmd.linked_idx != -1 -				  && gpg->fd_data_map[gpg->cmd.linked_idx].fd -				  != -1) -				{ -				  struct io_select_fd_s fds; -				  fds.fd = -				    gpg->fd_data_map[gpg->cmd.linked_idx].fd; -				  fds.for_read = 1; -				  fds.for_write = 0; -				  fds.opaque = NULL; -				  do -				    { -				      fds.signaled = 0; -				      _gpgme_io_select (&fds, 1, 1); -				      if (fds.signaled) -					_gpgme_data_inbound_handler -					  (gpg->cmd.linked_data, fds.fd); -				    } -				  while (fds.signaled); -				} - -			      /* XXX We must check if there are any -				 more fds active after removing this -				 one.  */ -			      (*gpg->io_cbs.remove) -				(gpg->fd_data_map[gpg->cmd.idx].tag); -			      gpg->cmd.fd = gpg->fd_data_map[gpg->cmd.idx].fd; -			      gpg->fd_data_map[gpg->cmd.idx].fd = -1; -			    } -                        }                      }                  }  	      /* To reuse the buffer for the next line we have to diff --git a/src/engine.c b/src/engine.c index b716ca24..b629bea7 100644 --- a/src/engine.c +++ b/src/engine.c @@ -596,8 +596,7 @@ _gpgme_engine_set_status_handler (engine_t engine,  gpgme_error_t  _gpgme_engine_set_command_handler (engine_t engine,  				   engine_command_handler_t fnc, -				   void *fnc_value, -				   gpgme_data_t linked_data) +				   void *fnc_value)  {    if (!engine)      return gpg_error (GPG_ERR_INV_VALUE); @@ -605,8 +604,7 @@ _gpgme_engine_set_command_handler (engine_t engine,    if (!engine->ops->set_command_handler)      return gpg_error (GPG_ERR_NOT_IMPLEMENTED); -  return (*engine->ops->set_command_handler) (engine->engine, -					      fnc, fnc_value, linked_data); +  return (*engine->ops->set_command_handler) (engine->engine, fnc, fnc_value);  }  gpgme_error_t diff --git a/src/engine.h b/src/engine.h index 8b692f2e..c512a252 100644 --- a/src/engine.h +++ b/src/engine.h @@ -78,8 +78,7 @@ void _gpgme_engine_set_status_handler (engine_t engine,  				       void *fnc_value);  gpgme_error_t _gpgme_engine_set_command_handler (engine_t engine,  						 engine_command_handler_t fnc, -						 void *fnc_value, -						 gpgme_data_t data); +						 void *fnc_value);  gpgme_error_t  _gpgme_engine_set_colon_line_handler (engine_t engine,  				      engine_colon_line_handler_t fnc, diff --git a/src/genkey.c b/src/genkey.c index 16484ecc..ffca7e8e 100644 --- a/src/genkey.c +++ b/src/genkey.c @@ -259,7 +259,7 @@ genkey_start (gpgme_ctx_t ctx, int synchronous, const char *parms,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -        (ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +        (ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)          return err;      } @@ -345,7 +345,7 @@ createkey_start (gpgme_ctx_t ctx, int synchronous,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -        (ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +        (ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)          return err;      } @@ -433,7 +433,7 @@ createsubkey_start (gpgme_ctx_t ctx, int synchronous,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -        (ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +        (ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)          return err;      } @@ -519,7 +519,7 @@ addrevuid_start (gpgme_ctx_t ctx, int synchronous, int extraflags,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -        (ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +        (ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)          return err;      } diff --git a/src/keysign.c b/src/keysign.c index c2fcabb5..5e497935 100644 --- a/src/keysign.c +++ b/src/keysign.c @@ -171,7 +171,7 @@ keysign_start (gpgme_ctx_t ctx, int synchronous,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -        (ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +        (ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)          return err;      } diff --git a/src/passwd.c b/src/passwd.c index 5bd67a52..6c03002b 100644 --- a/src/passwd.c +++ b/src/passwd.c @@ -151,7 +151,7 @@ passwd_start (gpgme_ctx_t ctx, int synchronous, gpgme_key_t key,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -        (ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +        (ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)          return err;      } @@ -449,7 +449,7 @@ sign_start (gpgme_ctx_t ctx, int synchronous, gpgme_data_t plain,    if (ctx->passphrase_cb)      {        err = _gpgme_engine_set_command_handler -	(ctx->engine, _gpgme_passphrase_command_handler, ctx, NULL); +	(ctx->engine, _gpgme_passphrase_command_handler, ctx);        if (err)  	return err;      } | 
