diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2010-04-22 18:37:47 +0100 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2010-04-22 18:47:51 +0100 |
| commit | 30a836d4568a8d270dff7f1036f3ec912ca9d9ae (patch) | |
| tree | 724860c8a18493e0d9847dccd2f382343aa3c740 /telepathy-logger/observer.c | |
| parent | bd67e2cfef7da52349b5a11d1591731665260ede (diff) | |
| download | telepathy-logger-30a836d4568a8d270dff7f1036f3ec912ca9d9ae.tar.gz | |
TplObserver: don't dereference self til it's been checked
Diffstat (limited to 'telepathy-logger/observer.c')
| -rw-r--r-- | telepathy-logger/observer.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/telepathy-logger/observer.c b/telepathy-logger/observer.c index 68f2d1e..e469de0 100644 --- a/telepathy-logger/observer.c +++ b/telepathy-logger/observer.c @@ -670,13 +670,11 @@ void tpl_observer_set_channel_factory (TplObserver *self, TplChannelFactory factory) { - TplObserverPriv *priv = self->priv; - - g_return_if_fail (factory != NULL); - g_return_if_fail (priv->channel_factory == NULL); g_return_if_fail (TPL_IS_OBSERVER (self)); + g_return_if_fail (factory != NULL); + g_return_if_fail (self->priv->channel_factory == NULL); - priv->channel_factory = factory; + self->priv->channel_factory = factory; } |