Address feedback from #1009

Add `has_nx_stack` and `has_nx_heap`

api/python/src/MachO/objects/pyBinary.cpp

@@ -644,6 +644,18 @@ void create<Binary>(nb::module_& m) {
         &Binary::page_size,
         "Return the binary's page size"_doc)
 
+    .def_prop_ro("has_nx_heap", &Binary::has_nx_heap,
+                 R"doc(
+                 Return True if the **heap** is flagged as non-executable. False
+                 otherwise.
+                 )doc"_doc)
+
+    .def_prop_ro("has_nx_stack", &Binary::has_nx_stack,
+                 R"doc(
+                 Return True if the **stack** is flagged as non-executable. False
+                 otherwise.
+                 )doc"_doc)
+
     .def("__getitem__",
         nb::overload_cast<LOAD_COMMAND_TYPES>(&Binary::operator[]),
         nb::rv_policy::reference_internal)

include/LIEF/MachO/Binary.hpp

@@ -452,6 +452,12 @@ class LIEF_API Binary : public LIEF::Binary  {
   //! Check if the binary uses ``NX`` protection
   bool has_nx() const override;
 
+  /// Return True if the **heap** is flagged as non-executable. False otherwise
+  bool has_nx_stack() const;
+
+  /// Return True if the **stack** is flagged as non-executable. False otherwise
+  bool has_nx_heap() const;
+
   //! ``true`` if the binary has an entrypoint.
   //!
   //! Basically for libraries it will return ``false``

src/MachO/Binary.cpp

@@ -207,14 +207,20 @@ bool Binary::is_pie() const {
   return header().has(HEADER_FLAGS::MH_PIE);
 }
 
-
 bool Binary::has_nx() const {
   if (!header().has(HEADER_FLAGS::MH_NO_HEAP_EXECUTION)) {
     LIEF_INFO("Heap could be executable");
   }
   return !header().has(HEADER_FLAGS::MH_ALLOW_STACK_EXECUTION);
 }
 
+bool Binary::has_nx_stack() const {
+  return !header().has(HEADER_FLAGS::MH_ALLOW_STACK_EXECUTION);
+}
+
+bool Binary::has_nx_heap() const {
+  return header().has(HEADER_FLAGS::MH_NO_HEAP_EXECUTION);
+}
 
 bool Binary::has_entrypoint() const {
   return has_main_command() || has_thread_command();