From patchwork Fri Dec 14 09:27:38 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aleksey Skripka X-Patchwork-Id: 11410 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id CAA8544DB43 for ; Fri, 14 Dec 2018 11:27:46 +0200 (EET) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id F03D268A990; Fri, 14 Dec 2018 11:27:46 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail.undev.ru (mail.undev.ru [213.232.193.241]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6330868A958 for ; Fri, 14 Dec 2018 11:27:40 +0200 (EET) Received: from localhost (localhost [127.0.0.1]) by mail.undev.ru (Postfix) with ESMTP id CD3D05F4294C for ; Fri, 14 Dec 2018 12:27:39 +0300 (MSK) Received: from mail.undev.ru ([127.0.0.1]) by localhost (mail.undev.ru [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id Al778TMhNT8a for ; Fri, 14 Dec 2018 12:27:38 +0300 (MSK) Received: from localhost (localhost [127.0.0.1]) by mail.undev.ru (Postfix) with ESMTP id B98FA5F431F2 for ; Fri, 14 Dec 2018 12:27:38 +0300 (MSK) X-Virus-Scanned: amavisd-new at undev.ru Received: from mail.undev.ru ([127.0.0.1]) by localhost (mail.undev.ru [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id rDJXeBr6bthZ for ; Fri, 14 Dec 2018 12:27:38 +0300 (MSK) Received: from [10.10.10.51] (unknown [188.244.38.45]) (Authenticated sender: caspy@undev.ru) by mail.undev.ru (Postfix) with ESMTPSA id 9BB0A5F4294C for ; Fri, 14 Dec 2018 12:27:38 +0300 (MSK) From: Aleksey Skripka Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\)) Date: Fri, 14 Dec 2018 12:27:38 +0300 References: <6095A590-81BB-4B94-A60F-DBF87B853ECF@undev.ru> To: FFmpeg development discussions and patches In-Reply-To: Message-Id: X-Mailer: Apple Mail (2.3445.102.3) X-Content-Filtered-By: Mailman/MimeDel 2.1.20 Subject: Re: [FFmpeg-devel] [PATCH] libavformat/hlsenc: fix broken -hls_flags +temp_file X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" greetings! fixed version. thanks. --- --- --- libavformat/hlsenc.c.orig 2018-12-14 09:25:06.541809226 +0000 +++ libavformat/hlsenc.c 2018-12-14 09:19:16.129377384 +0000 @@ -1348,7 +1348,7 @@ static int hls_window(AVFormatContext *s char temp_filename[1024]; int64_t sequence = FFMAX(hls->start_sequence, vs->sequence - vs->nb_entries); const char *proto = avio_find_protocol_name(s->url); - int use_temp_file = proto && !strcmp(proto, "file") && (s->flags & HLS_TEMP_FILE); + int use_temp_file = proto && !strcmp(proto, "file"); static unsigned warned_non_file; char *key_uri = NULL; char *iv_string = NULL; @@ -1463,7 +1463,7 @@ static int hls_start(AVFormatContext *s, AVFormatContext *vtt_oc = vs->vtt_avf; AVDictionary *options = NULL; const char *proto = avio_find_protocol_name(s->url); - int use_temp_file = proto && !strcmp(proto, "file") && (s->flags & HLS_TEMP_FILE); + int use_temp_file = proto && !strcmp(proto, "file") && (c->flags & HLS_TEMP_FILE); char *filename, iv_string[KEYSIZE*2 + 1]; int err = 0; @@ -2123,7 +2123,7 @@ static int hls_write_packet(AVFormatCont int stream_index = 0; int range_length = 0; const char *proto = avio_find_protocol_name(s->url); - int use_temp_file = proto && !strcmp(proto, "file") && (s->flags & HLS_TEMP_FILE); + int use_temp_file = proto && !strcmp(proto, "file") && (hls->flags & HLS_TEMP_FILE); uint8_t *buffer = NULL; VariantStream *vs = NULL; AVDictionary *options = NULL; @@ -2251,7 +2251,6 @@ static int hls_write_packet(AVFormatCont if (hls->flags & HLS_SINGLE_FILE) { ret = flush_dynbuf(vs, &range_length); if (ret < 0) { - av_free(old_filename); return ret; } vs->size = range_length; @@ -2269,20 +2268,12 @@ static int hls_write_packet(AVFormatCont return ret; } ff_format_io_close(s, &vs->out); - - // rename that segment from .tmp to the real one - if (use_temp_file && oc->url[0]) { - hls_rename_temp_file(s, oc); - av_free(old_filename); - old_filename = av_strdup(vs->avf->url); - - if (!old_filename) { - return AVERROR(ENOMEM); - } - } } } + if (use_temp_file && oc->url[0] && !(hls->flags & HLS_SINGLE_FILE)) + hls_rename_temp_file(s, oc); + old_filename = av_strdup(vs->avf->url); if (!old_filename) { return AVERROR(ENOMEM); @@ -2348,7 +2339,7 @@ static int hls_write_trailer(struct AVFo AVFormatContext *vtt_oc = NULL; char *old_filename = NULL; const char *proto = avio_find_protocol_name(s->url); - int use_temp_file = proto && !strcmp(proto, "file") && (s->flags & HLS_TEMP_FILE); + int use_temp_file = proto && !strcmp(proto, "file") && (hls->flags & HLS_TEMP_FILE); int i; int ret = 0; VariantStream *vs = NULL;