Tue, 05 Nov 2019 21:07:41 -0500
Add some NULL checks to silence scan-build.
| 2086 | 1 | /* This file is part of the Project Athena Zephyr Notification System. |
| 2 | * It contains source for the ZGetSubscriptions function. | |
| 3 | * | |
| 4 | * Created by: Robert French | |
| 5 | * | |
| 6 | * Copyright (c) 1987 by the Massachusetts Institute of Technology. | |
| 7 | * For copying and distribution information, see the file | |
|
31294
73607ab89c6f
Remove trailing whitespace
Richard Laager <rlaager@pidgin.im>
parents:
15435
diff
changeset
|
8 | * "mit-copyright.h". |
| 2086 | 9 | */ |
| 10 | ||
|
8792
b0645c9dc276
[gaim-migrate @ 9554]
Christian Hammond <chipx86@chipx86.com>
parents:
2086
diff
changeset
|
11 | #include "internal.h" |
| 2086 | 12 | |
|
12424
af82a40f2488
[gaim-migrate @ 14731]
Richard Laager <rlaager@pidgin.im>
parents:
11105
diff
changeset
|
13 | /* Prototype for -Wmissing-prototypes */ |
|
af82a40f2488
[gaim-migrate @ 14731]
Richard Laager <rlaager@pidgin.im>
parents:
11105
diff
changeset
|
14 | Code_t ZGetSubscriptions(ZSubscription_t *subscription, int *numsubs); |
|
af82a40f2488
[gaim-migrate @ 14731]
Richard Laager <rlaager@pidgin.im>
parents:
11105
diff
changeset
|
15 | |
|
af82a40f2488
[gaim-migrate @ 14731]
Richard Laager <rlaager@pidgin.im>
parents:
11105
diff
changeset
|
16 | Code_t ZGetSubscriptions(ZSubscription_t *subscription, int *numsubs) |
| 2086 | 17 | { |
|
39832
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
18 | int i; |
|
31294
73607ab89c6f
Remove trailing whitespace
Richard Laager <rlaager@pidgin.im>
parents:
15435
diff
changeset
|
19 | |
|
39832
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
20 | if (!__subscriptions_list) { |
|
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
21 | return (ZERR_NOSUBSCRIPTIONS); |
|
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
22 | } |
| 2086 | 23 | |
|
39832
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
24 | if (__subscriptions_next == __subscriptions_num) { |
|
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
25 | return (ZERR_NOMORESUBSCRIPTIONS); |
|
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
26 | } |
|
31294
73607ab89c6f
Remove trailing whitespace
Richard Laager <rlaager@pidgin.im>
parents:
15435
diff
changeset
|
27 | |
|
39828
7372deed606c
zephyr: Use GLib's min/max macros.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
31294
diff
changeset
|
28 | for (i = 0; i < MIN(*numsubs, __subscriptions_num - __subscriptions_next); |
|
7372deed606c
zephyr: Use GLib's min/max macros.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
31294
diff
changeset
|
29 | i++) { |
|
7372deed606c
zephyr: Use GLib's min/max macros.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
31294
diff
changeset
|
30 | subscription[i] = __subscriptions_list[i + __subscriptions_next]; |
|
7372deed606c
zephyr: Use GLib's min/max macros.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
31294
diff
changeset
|
31 | } |
| 2086 | 32 | |
|
39832
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
33 | if (__subscriptions_num - __subscriptions_next < *numsubs) { |
|
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
34 | *numsubs = __subscriptions_num - __subscriptions_next; |
|
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
35 | } |
| 2086 | 36 | |
|
39832
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
37 | __subscriptions_next += *numsubs; |
|
31294
73607ab89c6f
Remove trailing whitespace
Richard Laager <rlaager@pidgin.im>
parents:
15435
diff
changeset
|
38 | |
|
39832
df2dff0f1359
Fix misleading indentation warnings.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
39828
diff
changeset
|
39 | return ZERR_NONE; |
| 2086 | 40 | } |