[Remind-Fans] [PATCH 1/2] man/remind.1: -k no longer uses sprintf().

Simon Ruderich simon at ruderich.org
Fri Mar 15 13:05:43 EDT 2013


---
Hello,

I think it's unnecessary to warn the user about an issue which
can't happen (anymore).

Regards
Simon

 man/remind.1 | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/man/remind.1 b/man/remind.1
index 0801d60..486797e 100644
--- a/man/remind.1
+++ b/man/remind.1
@@ -237,10 +237,7 @@ pops up a window and displays its invocation arguments.  You could use:
 to have all of your \fBMSG\fR-type reminders processed using xmessage.
 .PP
 A word of warning:  It is very easy to spawn dozens of xmessage processes
-with the above technique.  So be very careful.  Also, the \fIcmd\fR is passed
-as an argument to \fBsprintf()\fR.  If you use formatting directives other
-than %s, or use more than one %s directive, there's a good chance that
-you'll crash \fBRemind\fR.  Finally, because all shell and whitespace
+with the above technique.  So be very careful.  Because all shell and whitespace
 characters are escaped, the program you execute with the \fB\-k\fR
 option must be prepared to handle the entire message as a single argument.
 .RE
-- 
1.8.2

-- 
+ privacy is necessary
+ using gnupg http://gnupg.org
+ public key id: 0x92FEFDB7E44C32F9
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.roaringpenguin.com/pipermail/remind-fans/attachments/20130315/1b898f86/attachment.pgp>


More information about the Remind-fans mailing list