[Asterisk-code-review] stasis recording: Correct ast asprintf error checking (asterisk[13])
Jenkins2
asteriskteam at digium.com
Thu Jun 1 10:47:54 CDT 2017
Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/5739 )
Change subject: stasis_recording: Correct ast_asprintf error checking
......................................................................
stasis_recording: Correct ast_asprintf error checking
ASTERISK-27021 #close
Reported by: Tim Morgan
Change-Id: I0ac061f040093e806c3b1f4e2340864f3ce4dd75
---
M res/stasis_recording/stored.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
George Joseph: Looks good to me, approved
Jenkins2: Approved for Submit
Joshua Colp: Looks good to me, but someone else must approve
diff --git a/res/stasis_recording/stored.c b/res/stasis_recording/stored.c
index ab05687..3ad7d82 100644
--- a/res/stasis_recording/stored.c
+++ b/res/stasis_recording/stored.c
@@ -157,7 +157,7 @@
return 0;
}
- if (ast_asprintf(&data->file_with_ext, "%s/%s", dir_name, filename)) {
+ if (ast_asprintf(&data->file_with_ext, "%s/%s", dir_name, filename) < 0) {
return -1;
}
@@ -252,7 +252,7 @@
return 0;
}
- if (ast_asprintf(&filepath, "%s/%s", dir_name, filename)) {
+ if (ast_asprintf(&filepath, "%s/%s", dir_name, filename) < 0) {
return -1;
}
--
To view, visit https://gerrit.asterisk.org/5739
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0ac061f040093e806c3b1f4e2340864f3ce4dd75
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
More information about the asterisk-code-review
mailing list