diff options
| author |  Stefan Richter
| 2010-06-20 18:03:33 +0200 |
|---|---|---|
| committer | Stefan Richter
| 2010-09-07 11:26:56 +0200 |
| commit | a29d69af5d8fa981722a92a058c24f4af4059702 (patch) | |
| tree | 4a111bae6fbb5f9f900c333d3a6106166978a904 /src/iso.c | |
| parent | tools/dumpiso: Add write() return code checks, fix harmless format string bug (diff) | |
Do not use a random FW_CDEV_VERSION as our implemented ABI version
Since linux/firewire-cdev.h header file and libraw1394 sources are
distributed separately, it is wrong to fill in a constant from that
header into the FW_CDEV_IOC_GET_INFO ioctl as the ABI version which
libraw1394 supports. This may not be forward compatible if an old
libraw1394 is compiled with a new kernel header and ran on top of a
kernel that implements new features that require a compatible
userland.
OK, the damage is already done in released versions of libraw1394.
Hence the FW_CDEV_VERSION of the kernel header file is not going to
be updated anymore in future kernel versions. (Only the version
internally to firewire-core will be incremented further.)
But let's remove the buggy usage of FW_CDEV_VERSION nevertheless.
Developers of other firewire-cdev client programs might look at
libraw1394 sources. The libraw1394 sources should not teach them
how to do it wrong.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'src/iso.c')
0 files changed, 0 insertions, 0 deletions