write/xcoff: add support.

Cargo.toml

@@ -3,7 +3,7 @@ name = "object"
 version = "0.30.0"
 edition = "2018"
 exclude = ["/.github", "/testfiles"]
-keywords = ["object", "elf", "mach-o", "pe", "coff"]
+keywords = ["object", "elf", "mach-o", "pe", "coff", "xcoff"]
 license = "Apache-2.0 OR MIT"
 repository = "https://github.com/gimli-rs/object"
 description = "A unified interface for reading and writing object file formats."

src/common.rs

@@ -467,4 +467,9 @@ pub enum SymbolFlags<Section> {
         /// `Number` field in the auxiliary symbol for the section.
         associative_section: Option<Section>,
     },
+    /// XCOFF symbol flags.
+    Xcoff {
+        /// `n_sclass` in the XCOFF symbol.
+        n_sclass: u8,
+    },
 }

src/read/xcoff/symbol.rs

@@ -313,19 +313,23 @@ impl<'data, 'file, Xcoff: FileHeader, R: ReadRef<'data>> ObjectSymbol<'data>
     }
 
     fn kind(&self) -> SymbolKind {
-        match self.symbol.n_sclass() {
-            xcoff::C_FILE => SymbolKind::File,
-            xcoff::C_NULL => SymbolKind::Null,
-            _ => self
-                .file
-                .section_by_index(SectionIndex((self.symbol.n_scnum() - 1) as usize))
-                .map(|section| match section.kind() {
-                    SectionKind::Data | SectionKind::UninitializedData => SymbolKind::Data,
-                    SectionKind::UninitializedTls | SectionKind::Tls => SymbolKind::Tls,
-                    SectionKind::Text => SymbolKind::Text,
-                    _ => SymbolKind::Unknown,
-                })
-                .unwrap_or(SymbolKind::Unknown),
+        if self.symbol.has_aux_csect() {
+            SymbolKind::Section
+        } else {
+            match self.symbol.n_sclass() {
+                xcoff::C_FILE => SymbolKind::File,
+                xcoff::C_NULL => SymbolKind::Null,
+                _ => self
+                    .file
+                    .section_by_index(SectionIndex((self.symbol.n_scnum() - 1) as usize))
+                    .map(|section| match section.kind() {
+                        SectionKind::Data | SectionKind::UninitializedData => SymbolKind::Data,
+                        SectionKind::UninitializedTls | SectionKind::Tls => SymbolKind::Tls,
+                        SectionKind::Text => SymbolKind::Text,
+                        _ => SymbolKind::Unknown,
+                    })
+                    .unwrap_or(SymbolKind::Unknown),
+            }
         }
     }
 
@@ -408,7 +412,9 @@ impl<'data, 'file, Xcoff: FileHeader, R: ReadRef<'data>> ObjectSymbol<'data>
 
     #[inline]
     fn flags(&self) -> SymbolFlags<SectionIndex> {
-        SymbolFlags::None
+        SymbolFlags::Xcoff {
+            n_sclass: self.symbol.n_sclass(),
+        }
     }
 }
 

src/write/mod.rs

@@ -29,6 +29,9 @@ mod macho;
 #[cfg(feature = "pe")]
 pub mod pe;
 
+#[cfg(feature = "xcoff")]
+mod xcoff;
+
 mod string;
 pub use string::StringId;
 
@@ -218,6 +221,8 @@ impl<'a> Object<'a> {
             BinaryFormat::Elf => self.elf_section_info(section),
             #[cfg(feature = "macho")]
             BinaryFormat::MachO => self.macho_section_info(section),
+            #[cfg(feature = "xcoff")]
+            BinaryFormat::Xcoff => self.xcoff_section_info(section),
             _ => unimplemented!(),
         }
     }
@@ -243,7 +248,7 @@ impl<'a> Object<'a> {
 
     fn has_subsections_via_symbols(&self) -> bool {
         match self.format {
-            BinaryFormat::Coff | BinaryFormat::Elf => false,
+            BinaryFormat::Coff | BinaryFormat::Elf | BinaryFormat::Xcoff => false,
             BinaryFormat::MachO => true,
             _ => unimplemented!(),
         }
@@ -506,6 +511,8 @@ impl<'a> Object<'a> {
             BinaryFormat::Elf => self.elf_fixup_relocation(&mut relocation)?,
             #[cfg(feature = "macho")]
             BinaryFormat::MachO => self.macho_fixup_relocation(&mut relocation),
+            #[cfg(feature = "xcoff")]
+            BinaryFormat::Xcoff => self.xcoff_fixup_relocation(&mut relocation),
             _ => unimplemented!(),
         };
         if addend != 0 {
@@ -574,6 +581,8 @@ impl<'a> Object<'a> {
             BinaryFormat::Elf => self.elf_write(buffer),
             #[cfg(feature = "macho")]
             BinaryFormat::MachO => self.macho_write(buffer),
+            #[cfg(feature = "xcoff")]
+            BinaryFormat::Xcoff => self.xcoff_write(buffer),
             _ => unimplemented!(),
         }
     }
@@ -894,6 +903,8 @@ pub enum Mangling {
     Elf,
     /// Mach-O symbol mangling.
     MachO,
+    /// Xcoff symbol mangling.
+    Xcoff,
 }
 
 impl Mangling {
@@ -904,14 +915,15 @@ impl Mangling {
             (BinaryFormat::Coff, _) => Mangling::Coff,
             (BinaryFormat::Elf, _) => Mangling::Elf,
             (BinaryFormat::MachO, _) => Mangling::MachO,
+            (BinaryFormat::Xcoff, _) => Mangling::Xcoff,
             _ => Mangling::None,
         }
     }
 
     /// Return the prefix to use for global symbols.
     pub fn global_prefix(self) -> Option<u8> {
         match self {
-            Mangling::None | Mangling::Elf | Mangling::Coff => None,
+            Mangling::None | Mangling::Elf | Mangling::Coff | Mangling::Xcoff => None,
             Mangling::CoffI386 | Mangling::MachO => Some(b'_'),
         }
     }