diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2013-09-18 14:07:57 +0100 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2013-09-19 14:25:33 +0100 |
| commit | cc2af09f608511bf316e94e4462ff041406c455b (patch) | |
| tree | 4211c5939a6ee36d0553492b9d79072837dea51c /src/mcd-dispatcher.c | |
| parent | fe1e8ce4b830f665fbe098b1928969d26c553441 (diff) | |
| download | telepathy-mission-control-cc2af09f608511bf316e94e4462ff041406c455b.tar.gz | |
Flag printf-ish functions with G_GNUC_PRINTF and fix resulting warnings
In particular, mcd-account-manager-default could crash when migrating
an account, since it would dereference arbitrary stack contents
as a pointer-to-string.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=69542
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Diffstat (limited to 'src/mcd-dispatcher.c')
| -rw-r--r-- | src/mcd-dispatcher.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mcd-dispatcher.c b/src/mcd-dispatcher.c index caa70076..5fea7f27 100644 --- a/src/mcd-dispatcher.c +++ b/src/mcd-dispatcher.c @@ -2051,7 +2051,7 @@ try_delegating (ChannelToDelegate *to_delegate) g_strdup (mcd_channel_get_object_path (to_delegate->channel)), v); - DEBUG ("...but failed to delegate it: %s", + DEBUG ("...but failed to delegate %s: %s", mcd_channel_get_object_path (to_delegate->channel), to_delegate->error->message); |