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 032064025C for ; Mon, 20 Dec 2021 00:55:58 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 814B668AEEA; Mon, 20 Dec 2021 02:55:56 +0200 (EET) Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-oln040092075084.outbound.protection.outlook.com [40.92.75.84]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 8917368AD86 for ; Mon, 20 Dec 2021 02:55:50 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=feR7JFIxSyOE+OGJJDUXcIQnYZBkwSkWfn8SaXoAYuTZJGJCdxFxag/XUcSr+zOyL4q6IDJy1yVEiaqgK/GbuLDlAPYQqBBUYWsfjJWNCAxAPbQr3T9k2Y9EZQPTKNuxOrLcTR5u/58tgHj2EtKs3h84q7sFUuo6pyBYTGTGY6h77diLI5eIWV7naj9AloJLb3A+Ea3l56vKe4QePZ6hmYox5f1bfW82eNxOrB6XclOIRq1YQku65NC1dSMykPrAbblbnbHqLmka539cqMHSVukSrXNcn3+AymEvKhEc/J+Wmryz8d3anpmKddG2p51BdozlQbtcSQhMEaq08/kKow== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=iRQZYrLIlx9lu5oYKGuEqYKvAEmKs/OoiGayLPyFPDk=; b=Btw4hFaavLvy1WWNKuz73ZiIlIv/h64zlP4O0mmhuh7JLx55ce/27yWGbSw4WYncQOBaaNf1WgCCJTZftwFRj9SJSVv5hdBDQa71o9AtlugPpL2BJQcwEaO92yhFgqyg40+8cCYEA0S1QOtr+dEk62hQdmXCuROU+OT89tM2bFSsurNU2Qkpd/zuK2J0rwIq3mrYBqDaxVtE2gd4MSDyleyqqJAlAIvkQZemS3EbOEpCAEd6ZP7Aoin8VSTf79B2cXlJhrDbLtRIglC5MJ2E92RIf20KFuIPCkJvWovA6HJ2ujKUXmZgAHPGFJPRXIDT41HvILbEBcxCR0BGdv+FaA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=iRQZYrLIlx9lu5oYKGuEqYKvAEmKs/OoiGayLPyFPDk=; b=J8YP74tKfbOA6HQJyCWEcu5meywA9W5uXNqtuV+KVAIuWcT7EbrB56wZGepLQ82wcz82Qr+UNVMVo1lYjnPUYFGAU6WNf5/C9lKnBQ7km/p8/tK35Oy94fp3LiesX+hxw1/g95IUlZRKVNHhOu0yMEXEKoX80ET2C7Oe5m/kVlivlLquJBcbAyUqbiR4GlZMxwCq0wtvnH3/ojBDdGP8jbC8HtKi7mGRfX36kfxNYdl8eEP06smlC4tdCNTCh37/KTFnyHGMbyQmh/SOkZMs8uLr9j+n9dSDV0UjgBNU6CBrHx5/WlzihRwfZ9T3JDqptXRAk7JbU4QY+l9Ar2qHbw== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by AS8PR03MB6984.eurprd03.prod.outlook.com (2603:10a6:20b:29e::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4801.17; Mon, 20 Dec 2021 00:55:49 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::f0dc:92f5:6bc2:45ca]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::f0dc:92f5:6bc2:45ca%6]) with mapi id 15.20.4801.020; Mon, 20 Dec 2021 00:55:48 +0000 Message-ID: Date: Mon, 20 Dec 2021 01:55:47 +0100 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20211219192134.1296-1-dcnieho@gmail.com> <20211219192134.1296-9-dcnieho@gmail.com> From: Andreas Rheinhardt In-Reply-To: <20211219192134.1296-9-dcnieho@gmail.com> X-TMN: [oBYUlDBoFn/ANDT9yZaYt/XPttgZmfRs] X-ClientProxiedBy: AM9P195CA0016.EURP195.PROD.OUTLOOK.COM (2603:10a6:20b:21f::21) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <8e0b7795-1dab-8c69-3f62-b53f79c81d74@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 4a80bf29-3f53-46fc-9ddc-08d9c3537676 X-MS-TrafficTypeDiagnostic: AS8PR03MB6984:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: oRy1dLkbvwH+P/xSotORzUgX58o8h5sSNaHDfuGRZYidwBFPrjxTe9rNfUdMRTwaDcjepfu5jOavPr9VsEXHlreQM69sdaBV8DQ0MNhDdY5T8nOxRiaIsVvAvYzHZGkkRzLoMfBZVPqLvx5lgdEh9Bmf79QeExUCVM95G7z+DUS1dztHLCnWEWSUD1xYd+nzSLGYalmQdhuOW7qwNRV58xxykSUxVpGor2UNPBgCXuaE2+m2COt0y576IlczkDufmI3Vzo1a/QJpSeN3Tzx2Mm9tjJA/t96Kxp1tz4WRXkka7jsb44IiNwMjG7jd9JUhP/loWCcXBJrjjXcZh5ZmI8ZqRHqtL6m3D7ke+GU9mkGP2RK5KnImQLJPcgIywne+WcWgubw3evdPN6VxCL4UViHVIbqV1h/dj/UVS+UzfnfIGok+UeqYqgXuiKdo0I3zu1eUoGawekb3wIoKH5szvVHDHoPi9zjQ6g+yjKyhfvFK9ZnI0F5b45Pwk5QihRU4xSAafefWgp3irmoCGKVlrKBe6WcgKi78WfJ0c6sL7MnWDWHHQqxPZqWdPDYyNSuCU+vq8dI+2rnTSUlKtV13WA== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?c2FEVzlMWGNlUmdYMFVhZUxVdHh3bEJWR1pGU2NydXRnSm5RaWRCODNrYUxP?= =?utf-8?B?Wi8yTEZnN3FuZ3FOakt5UUVCSnB4dEtVMDdnREpNdFFjclZDNGMwUVlac1d1?= =?utf-8?B?dm5NWFV1Vm1LdklvZWIvWW5kSjVGZFg5c2ZHSTNNOWJrcmZJZ0Q4MldKeVlS?= =?utf-8?B?amJwdjdVNVRITktKcUhTNFVPcUV2T0xiUmcycWRFYXd3bTBoTnJHV2tWQkhw?= =?utf-8?B?S1haeU13Z1ZkNlRVZ2R1bDlHeUwwTEdRZDkwRk1jV2dZemN3ZFE2bkJWMGM4?= =?utf-8?B?Mi9IMVZ0SUV4MnVTa0FuQmJmUmk1WjFEWDhLQUVtTjZ4K3pHMDd0RGY5anI5?= =?utf-8?B?QStHcG5lZU1odEI3emJrU1FpYmlDSFQ1RWNDWko2TTVkVG5LeWRhRkEzUW43?= =?utf-8?B?MVpHdVRBWWQ5cTNUaUZpS0tpdVB0ZklDRDJqU3RWVlNyaEkvaDVMTWNWZ0d2?= =?utf-8?B?ajZmQUJBWmVDdmRtNVJINUpNNVBEOTQ1a2szbGZBN0ZhR3VnMkZ2VnRvME9m?= =?utf-8?B?bzBra3E3NUdTMlI1OEZlVnRpdEVmalhaa3ZMVng3a2syd2dublVnY21NUWU0?= =?utf-8?B?QmQ5QWZ6cDBWanVnRkFybG5IVzllSklhdm5wNUNuQ2lTR1h1RVN6R2RCcVl2?= =?utf-8?B?MXRYaHRsUEUxRm00UE8zYnJjalhZampOSGxpUWQ2OC9maFVMemJlNmVDV3Ar?= =?utf-8?B?TjlMZlZnZmdMb3dtQnVpL0x3Wi9FNGJCUjNDQ3E0MDdWNmNNK3FIWDk3SnUr?= =?utf-8?B?K1NvZmZCMEM0b3VENm1SVWdHa0UvSlNjQVNqRGl0VkExcVVwU09nbmd6cGpi?= =?utf-8?B?QTlHZGlOM0Exc2pqQ05SV0YvVnJZSjVYSkNCZlFIUTZQVU1OUVFsQkRmTFl5?= =?utf-8?B?ZkRjb24rdTBRS0pOS3diMnJobGdrT2FIVkJva1dwcTE4ZllVbW5ybHI1em9n?= =?utf-8?B?eTNHL0dmNVpqREFOb2luWnlmTnphVDBaU2w5YkVSY3lBWE5tZXhWeVY3ZlZX?= =?utf-8?B?TWdjWWRidjFDYWZ4VWNDL29vaGNkeGlaT3ZaVWZUNGxTQXAzWWl0QWtsSlVZ?= =?utf-8?B?SkRuN3dyQTE1UTFwQ0JTU3ZnbzEyTkpob1kya1FXVVM4ZWJVYjFkMVRFYTFj?= =?utf-8?B?UTJrNUY4M2szbTJON1k0alBNUWVYRnJPK2ZaU0JoWmRxYytuZmY2MGY5UkRI?= =?utf-8?B?ZmFkMXBUY2hib0IvNXFVTG9tM1B6NFBSL0QzRG9mMWkrQnhzblNGWWRvdlVu?= =?utf-8?B?OW9JcmRCMVorZldFYWlOdnZ3bTdHTEdQcHdUK2NqSEd5djhKUT09?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4a80bf29-3f53-46fc-9ddc-08d9c3537676 X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Dec 2021 00:55:48.8540 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR03MB6984 Subject: Re: [FFmpeg-devel] [PATCH v5 08/13] avdevice/dshow: list_devices: show media type(s) per device 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: Diederick Niehorster: > the list_devices option of dshow didn't indicate whether a specific > device provides audio or video output. This patch iterates through all > media formats of all pins exposed by the device to see what types it > provides for capture, and prints this to the console for each device. > Importantly, this now allows to find devices that provide both audio and > video, and devices that provide neither. > > Signed-off-by: Diederick Niehorster > --- > libavdevice/dshow.c | 103 +++++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 98 insertions(+), 5 deletions(-) > > diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c > index 8c257ca8e7..f25537db5c 100644 > --- a/libavdevice/dshow.c > +++ b/libavdevice/dshow.c > @@ -197,6 +197,79 @@ fail: > return; > } > > +static void dshow_get_device_media_types(AVFormatContext *avctx, enum dshowDeviceType devtype, > + enum dshowSourceFilterType sourcetype, IBaseFilter *device_filter, > + enum AVMediaType **media_types, int *nb_media_types) > +{ > + struct dshow_ctx *ctx = avctx->priv_data; > + IEnumPins *pins = 0; > + IPin *pin; > + int has_audio = 0, has_video = 0; > + > + if (IBaseFilter_EnumPins(device_filter, &pins) != S_OK) > + return; > + > + while (IEnumPins_Next(pins, 1, &pin, NULL) == S_OK) { > + IKsPropertySet *p = NULL; > + PIN_INFO info = { 0 }; > + GUID category; > + DWORD r2; > + IEnumMediaTypes *types = NULL; > + AM_MEDIA_TYPE *type; > + > + if (IPin_QueryPinInfo(pin, &info) != S_OK) > + goto next; > + IBaseFilter_Release(info.pFilter); > + > + if (info.dir != PINDIR_OUTPUT) > + goto next; > + if (IPin_QueryInterface(pin, &IID_IKsPropertySet, (void **) &p) != S_OK) > + goto next; > + if (IKsPropertySet_Get(p, &ROPSETID_Pin, AMPROPERTY_PIN_CATEGORY, > + NULL, 0, &category, sizeof(GUID), &r2) != S_OK) > + goto next; > + if (!IsEqualGUID(&category, &PIN_CATEGORY_CAPTURE)) > + goto next; > + > + if (IPin_EnumMediaTypes(pin, &types) != S_OK) > + goto next; > + > + // enumerate media types exposed by pin > + // NB: don't know if a pin can expose both audio and video, check 'm all to be safe > + IEnumMediaTypes_Reset(types); > + while (IEnumMediaTypes_Next(types, 1, &type, NULL) == S_OK) { > + if (IsEqualGUID(&type->majortype, &MEDIATYPE_Video)) { > + has_video = 1; > + } else if (IsEqualGUID(&type->majortype, &MEDIATYPE_Audio)) { > + has_audio = 1; > + } > + CoTaskMemFree(type); > + } > + > + next: > + if (types) > + IEnumMediaTypes_Release(types); > + if (p) > + IKsPropertySet_Release(p); > + if (pin) > + IPin_Release(pin); > + } > + > + IEnumPins_Release(pins); > + > + if (has_audio || has_video) { > + int nb_types = has_audio + has_video; > + *media_types = av_malloc_array(nb_types, sizeof(enum AVMediaType)); You are allocating a list of at most two entries; why not put an array of that many entries on the stack in dshow_cycle_devices() and write them there. This would get rid of the allocations, the frees and would fix the problem that you do not forward the error in case of allocation failure. (The required size of said array should of course be defined by a macro.) > + if (*media_types) { > + if (has_audio) > + (*media_types)[0] = AVMEDIA_TYPE_AUDIO; > + if (has_video) > + (*media_types)[0 + has_audio] = AVMEDIA_TYPE_VIDEO; > + *nb_media_types = nb_types; > + } > + } > +} > + > /** > * Cycle through available devices using the device enumerator devenum, > * retrieve the device with type specified by devtype and return the > @@ -242,6 +315,8 @@ dshow_cycle_devices(AVFormatContext *avctx, ICreateDevEnum *devenum, > LPOLESTR olestr = NULL; > LPMALLOC co_malloc = NULL; > AVDeviceInfo *device = NULL; > + enum AVMediaType *media_types = NULL; > + int nb_media_types = 0; > int i; > > r = CoGetMalloc(1, &co_malloc); > @@ -286,6 +361,12 @@ dshow_cycle_devices(AVFormatContext *avctx, ICreateDevEnum *devenum, > // success, loop will end now > } > } else { > + // get media types exposed by pins of device > + if (IMoniker_BindToObject(m, 0, 0, &IID_IBaseFilter, (void* ) &device_filter) == S_OK) { > + dshow_get_device_media_types(avctx, devtype, sourcetype, device_filter, &media_types, &nb_media_types); > + IBaseFilter_Release(device_filter); > + device_filter = NULL; > + } > if (device_list) { > device = av_mallocz(sizeof(AVDeviceInfo)); > if (!device) > @@ -306,12 +387,26 @@ dshow_cycle_devices(AVFormatContext *avctx, ICreateDevEnum *devenum, > device = NULL; // copied into array, make sure not freed below > } > else { > - av_log(avctx, AV_LOG_INFO, " \"%s\"\n", friendly_name); > - av_log(avctx, AV_LOG_INFO, " Alternative name \"%s\"\n", unique_name); > + av_log(avctx, AV_LOG_INFO, "\"%s\"", friendly_name); > + if (nb_media_types > 0 && media_types) { You are checking both the number of media types as well as the pointer; the second check is redundant if you never set the number inconsistently (which you don't). > + const char* media_type = av_get_media_type_string(media_types[0]); > + av_log(NULL, AV_LOG_INFO, " (%s", media_type ? media_type : "unknown"); av_log(NULL should be avoided. Use a proper logcontext. > + for (int i = 1; i < nb_media_types; ++i) { > + media_type = av_get_media_type_string(media_types[i]); > + av_log(NULL, AV_LOG_INFO, ", %s", media_type ? media_type : "unknown"); > + } > + av_log(NULL, AV_LOG_INFO, ")"); > + } else { > + av_log(NULL, AV_LOG_INFO, " (none)"); > + } > + av_log(avctx, AV_LOG_INFO, "\n"); > + av_log(avctx, AV_LOG_INFO, " Alternative name \"%s\"\n", unique_name); > } > } > > -fail1: > + fail1: > + if (media_types) > + av_freep(&media_types); > if (device) { > if (device->device_name) > av_freep(&device->device_name); > @@ -1186,9 +1281,7 @@ static int dshow_read_header(AVFormatContext *avctx) > } > > if (ctx->list_devices) { > - av_log(avctx, AV_LOG_INFO, "DirectShow video devices (some may be both video and audio devices)\n"); > dshow_cycle_devices(avctx, devenum, VideoDevice, VideoSourceDevice, NULL, NULL, NULL); > - av_log(avctx, AV_LOG_INFO, "DirectShow audio devices\n"); > dshow_cycle_devices(avctx, devenum, AudioDevice, AudioSourceDevice, NULL, NULL, NULL); > ret = AVERROR_EXIT; > goto error; > _______________________________________________ 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".