From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 9132746BD9 for ; Tue, 4 Jul 2023 23:26:37 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 7CFBB68C640; Wed, 5 Jul 2023 02:26:34 +0300 (EEST) Received: from mail-oi1-f172.google.com (mail-oi1-f172.google.com [209.85.167.172]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 5E01B68C618 for ; Wed, 5 Jul 2023 02:26:28 +0300 (EEST) Received: by mail-oi1-f172.google.com with SMTP id 5614622812f47-3942c6584f0so4143076b6e.3 for ; Tue, 04 Jul 2023 16:26:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1688513186; x=1691105186; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=ivzggAa5WKHkoQORz9F2hrpXenvLKGO1sO7rA55WgU8=; b=LPWXuoSy1Y28dhauEUhLSoCbLhge2aCEkWlthbCM2N8BGUDUGgT0AetGwTQIWE23AL VabbXCbGJspXxjHUg9bbyXE6wpxGRkypnfbYqh/OUZtA0RuKbef/6hn33a+CvsNTxmq0 pFeJv1DPZ4hHxQ/NvuM+hqudgq2kvy1RMUmsTbeh6X0i7T1Rd74k3r5s0Bp1jXbm109G Vl5l/JWww25yljuNr9YPcVy0pef8A0S0l3b3Sh1jnH4g9lLCjf03RpEbGT6WPbhSo3Ez K+EHlpgIONaBMJl4uq49eK51bzyqhzGvb5NHDj3W7mAn2ucbP6ADIRyZEIf/zagVFRFF KlRg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1688513186; x=1691105186; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ivzggAa5WKHkoQORz9F2hrpXenvLKGO1sO7rA55WgU8=; b=P11O5X7qngAeItmMeHC1O3FyPVANwvChQPvxIPSWt23yhYM1UfjJpgAFIt1syqzf8M kEd5H49KDNotzghnKZylzPq/hM+icShX+ntPVDKRD+6mwJ6uXbAVTEPO4bHxAsUfNons JB0om6dWkX/j0nVaI1i1pfSbI81fOn6vnaDbelDLR8CFidWzvHkYUfXKdJQLpwau7y7r jHUPV04NTkbE0zfIrsgEDwPkLZ/rFo5NXyc1DWmd1kKsdjP47jmgwktbtQ8508qoeaEO ZlJxVDeiFlsMPkhs9Avt5j19D36Rvt/xJ3QjtYbDc1bVWZhO9fGZSoWs/rogJYQ01vYF 4a4Q== X-Gm-Message-State: ABy/qLb25PxhBG/xLiqtZOr6cHKYFXUedtqT6iG7/fIwz1E/mWu0BFDl hJ/7RuxXgc3+o3NGZV9YDQM0xi5HC2U= X-Google-Smtp-Source: APBJJlEp09yE/Bh1jhEAwCRNPrLeqtZDUoWlcgRU5Kfcmzll+qPpK4chz5rvEbKCXc+k3sy/NvSebw== X-Received: by 2002:a05:6871:4795:b0:1b3:8285:153e with SMTP id qa21-20020a056871479500b001b38285153emr8909030oab.44.1688513185783; Tue, 04 Jul 2023 16:26:25 -0700 (PDT) Received: from localhost.localdomain (host197.190-225-105.telecom.net.ar. [190.225.105.197]) by smtp.gmail.com with ESMTPSA id er28-20020a056870c89c00b0019ed19a8659sm14507303oab.8.2023.07.04.16.26.24 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 04 Jul 2023 16:26:25 -0700 (PDT) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Tue, 4 Jul 2023 20:26:12 -0300 Message-ID: <20230704232614.2785-1-jamrial@gmail.com> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH v3 1/3] avutil/random_seed: use fread() in read_random() X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 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 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: This ensures the requested amount of bytes is read. Also remove /dev/random as it's no longer necessary. Signed-off-by: James Almer --- libavutil/random_seed.c | 23 ++++++++++++----------- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/libavutil/random_seed.c b/libavutil/random_seed.c index 66dd504ef0..a51149235b 100644 --- a/libavutil/random_seed.c +++ b/libavutil/random_seed.c @@ -49,17 +49,20 @@ static int read_random(uint32_t *dst, const char *file) { #if HAVE_UNISTD_H - int fd = avpriv_open(file, O_RDONLY); - int err = -1; + FILE *fp = avpriv_fopen_utf8(file, "r"); + size_t err; - if (fd == -1) - return -1; - err = read(fd, dst, sizeof(*dst)); - close(fd); + if (!fp) + return AVERROR_UNKNOWN; + err = fread(dst, 1, sizeof(*dst), fp); + fclose(fp); - return err; + if (err != sizeof(*dst)) + return AVERROR_UNKNOWN; + + return 0; #else - return -1; + return AVERROR(ENOSYS); #endif } @@ -138,9 +141,7 @@ uint32_t av_get_random_seed(void) return arc4random(); #endif - if (read_random(&seed, "/dev/urandom") == sizeof(seed)) - return seed; - if (read_random(&seed, "/dev/random") == sizeof(seed)) + if (!read_random(&seed, "/dev/urandom")) return seed; return get_generic_seed(); } -- 2.41.0 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".