Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
htsp: initialize streaming_skip_t variable properly
  • Loading branch information
perexg committed Jun 2, 2015
1 parent f63239e commit caa4333
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/htsp_server.c
Expand Up @@ -2197,6 +2197,7 @@ htsp_method_skip(htsp_connection_t *htsp, htsmsg_t *in)

abs = htsmsg_get_u32_or_default(in, "absolute", 0);

memset(&skip, 0, sizeof(skip));
if(!htsmsg_get_s64(in, "time", &s64)) {
skip.type = abs ? SMT_SKIP_ABS_TIME : SMT_SKIP_REL_TIME;
skip.time = hs->hs_90khz ? s64 : ts_rescale_i(s64, 1000000);
Expand Down Expand Up @@ -2265,6 +2266,7 @@ htsp_method_live(htsp_connection_t *htsp, htsmsg_t *in)
if(hs == NULL)
return htsp_error("Requested subscription does not exist");

memset(&skip, 0, sizeof(skip));
skip.type = SMT_SKIP_LIVE;
tvhtrace("htsp-sub", "live");
subscription_set_skip(hs->hs_s, &skip);
Expand Down

3 comments on commit caa4333

@ksooo
Copy link
Contributor

@ksooo ksooo commented on caa4333 Jun 3, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this one serious wrt timeshifting functionality?

@perexg
Copy link
Contributor Author

@perexg perexg commented on caa4333 Jun 4, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, it's just to keep valgrind happy.

@ksooo
Copy link
Contributor

@ksooo ksooo commented on caa4333 Jun 4, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay. thx.

Please sign in to comment.