Skip to content

Commit

Permalink
[fix] fixed some bugs in stat module.
Browse files Browse the repository at this point in the history
  • Loading branch information
@winshining
winshining committed Sep 5, 2020
1 parent 671c353 commit 13be96d
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions ngx_rtmp_stat_module.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -711,6 +711,9 @@ ngx_rtmp_stat_live(ngx_http_request_t *r, ngx_chain_t ***lll,
NGX_RTMP_STAT(bbuf, ngx_snprintf(bbuf, sizeof(bbuf),
"%D", ctx->cs[1].timestamp -
ctx->cs[0].timestamp) - bbuf);
} else {
NGX_RTMP_STAT(bbuf, ngx_snprintf(bbuf, sizeof(bbuf),
"%D", 0) - bbuf);
}
NGX_RTMP_STAT_L("</avsync>\r\n");

Expand All @@ -736,6 +739,9 @@ ngx_rtmp_stat_live(ngx_http_request_t *r, ngx_chain_t ***lll,
NGX_RTMP_STAT(bbuf, ngx_snprintf(bbuf, sizeof(bbuf),
"%D", ctx->cs[1].timestamp -
ctx->cs[0].timestamp) - bbuf);
} else {
NGX_RTMP_STAT(bbuf, ngx_snprintf(bbuf, sizeof(bbuf),
"%D", 0) - bbuf);
}

NGX_RTMP_STAT_L(",\"timestamp\":");
Expand Down Expand Up @@ -1276,7 +1282,7 @@ ngx_rtmp_stat_application_recorders(ngx_http_request_t *r, ngx_chain_t ***lll,
flag[len++] = ',';
}

if (NGX_RTMP_MAX_URL - len > sizeof("\"video\"")) {
if (NGX_RTMP_MAX_URL - len >= sizeof("\"video\"")) {
*ngx_snprintf(flag + ngx_strlen(flag),
NGX_RTMP_MAX_URL - len,
"%s", "\"video\"") = 0;
Expand All @@ -1296,7 +1302,7 @@ ngx_rtmp_stat_application_recorders(ngx_http_request_t *r, ngx_chain_t ***lll,
flag[len++] = ',';
}

if (NGX_RTMP_MAX_URL - len > sizeof("\"audio\"")) {
if (NGX_RTMP_MAX_URL - len >= sizeof("\"audio\"")) {
*ngx_snprintf(flag + ngx_strlen(flag),
NGX_RTMP_MAX_URL - len,
"%s", "\"audio\"") = 0;
Expand All @@ -1316,7 +1322,7 @@ ngx_rtmp_stat_application_recorders(ngx_http_request_t *r, ngx_chain_t ***lll,
flag[len++] = ',';
}

if (NGX_RTMP_MAX_URL - len > sizeof("\"keyframes\"")) {
if (NGX_RTMP_MAX_URL - len >= sizeof("\"keyframes\"")) {
*ngx_snprintf(flag + ngx_strlen(flag),
NGX_RTMP_MAX_URL - len,
"%s", "\"keyframes\"") = 0;
Expand All @@ -1336,7 +1342,7 @@ ngx_rtmp_stat_application_recorders(ngx_http_request_t *r, ngx_chain_t ***lll,
flag[len++] = ',';
}

if (NGX_RTMP_MAX_URL - len > sizeof("\"manual\"")) {
if (NGX_RTMP_MAX_URL - len >= sizeof("\"manual\"")) {
*ngx_snprintf(flag + ngx_strlen(flag),
NGX_RTMP_MAX_URL - len,
"%s", "\"manual\"") = 0;
Expand Down

0 comments on commit 13be96d

Please sign in to comment.