Bug 454233 - file -s ids ext3 filesystem as mp3
file -s ids ext3 filesystem as mp3
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: file (Show other bugs)
rawhide
All Linux
low Severity low
: ---
: ---
Assigned To: Tomas Smetana
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-06 23:50 EDT by Jason Farrell
Modified: 2008-07-22 01:46 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-07-22 01:46:45 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Jason Farrell 2008-07-06 23:50:36 EDT
When using the new magic in file-4.24-4, it's no longer possible to id an ext3
block device as ext3 with 'file -s'

### with old magic (4.23):
[root@nano ~]# file -s /dev/mapper/vgr5-home
/dev/mapper/vgr5-home: Linux rev 1.0 ext3 filesystem data (needs journal
recovery) (large files)
### with new magic (4.24):
[root@nano dl]# file -s /dev/mapper/vgr5-home
/dev/mapper/vgr5-home: , 44.1 kHz, Stereo


After some poking around I've narrowed down the cause to an error in the updated
MP3 section:
--- 4.23-5.magic        2008-03-11 03:57:22.000000000 -0400
+++ 4.24-4.magic        2008-06-06 07:47:13.000000000 -0400
<snip>
@@ -592,22 +564,39 @@
 # modified to fully support MPEG ADTS

 # MP3, M1A
-0       beshort&0xFFFE  0xFFFA         MPEG ADTS, layer III, v1
+# modified by Joerg Jenderek
+# GRR the original test are too common for many DOS files
+# so don't accept as MP3 until we've tested the rate
+0       beshort&0xFFFE
 # rates
<snip>


Wrong number of fields. Must be >= 3. Works fine when using the old fields
(tested using ~/.magic).
Comment 1 Tomas Smetana 2008-07-21 07:21:21 EDT
I just pushed the new file-4.25 to rawhide and it seems to be working fine.  Can
you verify this please?  Thank you.
Comment 2 Jason Farrell 2008-07-21 10:26:36 EDT
Verified. Thanks.

Updated: file.x86_64 0:4.25-1.fc10 file-libs.x86_64 0:4.25-1.fc10
Complete!
[root@nano tmp]# file -s /dev/mapper/vgr5-home
/dev/mapper/vgr5-home: Linux rev 1.0 ext3 filesystem data (needs journal
recovery) (large files)
[root@nano tmp]# file -s /dev/mapper/vgr10-rootf9
/dev/mapper/vgr10-rootf9: Linux rev 1.0 ext4 filesystem data (needs journal
recovery) (extents) (large files)

Note You need to log in before you can comment on or make changes to this bug.